Commit ec337b2
tools: replace for loop with map()
Refs: #41406 (comment)
Co-authored-by: Tobias Nießen <tniessen@tnie.de>
PR-URL: #41451
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>1 parent fa84353 commit ec337b2
1 file changed
Lines changed: 1 addition & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -101,12 +101,8 @@ all = all.slice(0, apiStart.index + apiStart[0].length) + | |||
| 101 | 101 | fs.writeFileSync(new URL('./all.html', source), all, 'utf8'); | |
| 102 | 102 | ||
| 103 | 103 | // Validate all hrefs have a target. | |
| 104 | - const ids = new Set(); | ||
| 105 | 104 | const idRe = / id="([^"]+)"/g; | |
| 106 | - const idMatches = all.matchAll(idRe); | ||
| 107 | - for (const match of idMatches) { | ||
| 108 | - ids.add(match[1]); | ||
| 109 | - } | ||
| 105 | + const ids = new Set([...all.matchAll(idRe)].map((match) => match[1])); | ||
| 110 | 106 | ||
| 111 | 107 | const hrefRe = / href="#([^"]+)"/g; | |
| 112 | 108 | const hrefMatches = all.matchAll(hrefRe); | |
0 commit comments