Commit 2e38f0d
tools: remove superfluous regex in tools/doc/json.js
Remove noCallOrProp from tools/doc/json.js. It is a negative lookahead
that is only used in one regex where it is placed immediately before a
terminating $, thus rendering it meaningless.
PR-URL: #33998
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jamie Davis <davisjam@vt.edu>1 parent ee36c87 commit 2e38f0d
1 file changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -456,8 +456,6 @@ const maybeAncestors = r`(?:${id}\.?)*`; | |||
| 456 | 456 | ||
| 457 | 457 | const callWithParams = r`\([^)]*\)`; | |
| 458 | 458 | ||
| 459 | - const noCallOrProp = '(?![.[(])'; | ||
| 460 | - | ||
| 461 | 459 | const maybeExtends = `(?: +extends +${maybeAncestors}${classId})?`; | |
| 462 | 460 | ||
| 463 | 461 | /* eslint-disable max-len */ | |
@@ -478,7 +476,7 @@ const headingExpressions = [ | |||
| 478 | 476 | `^${maybeBacktick}${maybeAncestors}(${id})${callWithParams}${maybeBacktick}$`, 'i') }, | |
| 479 | 477 | ||
| 480 | 478 | { type: 'property', re: RegExp( | |
| 481 | - `^${maybeClassPropertyPrefix}${maybeBacktick}${ancestors}(${id})${maybeBacktick}${noCallOrProp}$`, 'i') }, | ||
| 479 | + `^${maybeClassPropertyPrefix}${maybeBacktick}${ancestors}(${id})${maybeBacktick}$`, 'i') }, | ||
| 482 | 480 | ]; | |
| 483 | 481 | /* eslint-enable max-len */ | |
| 484 | 482 | ||
0 commit comments