Reorder operations in getDescendantsByType helper for ~1% runtime red…

…uction.
DavidAnson · Oct 24, 2024 · 16512bb · 16512bb
1 parent e6e799d
commit 16512bb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/demo/markdownlint-browser.js b/demo/markdownlint-browser.js
@@ -843,9 +843,8 @@ function getBlockQuotePrefixText(tokens, lineNumber, count = 1) {
 function getDescendantsByType(parent, typePath) {
   let tokens = Array.isArray(parent) ? parent : [ parent ];
   for (const type of typePath) {
-    tokens = tokens
-      .flatMap((t) => t.children)
-      .filter((t) => Array.isArray(type) ? type.includes(t.type) : (type === t.type));
+    const predicate = (token) => Array.isArray(type) ? type.includes(token.type) : (type === token.type);
+    tokens = tokens.flatMap((t) => t.children.filter(predicate));
   }
   return tokens;
 }

diff --git a/helpers/micromark-helpers.cjs b/helpers/micromark-helpers.cjs
@@ -161,9 +161,8 @@ function getBlockQuotePrefixText(tokens, lineNumber, count = 1) {
 function getDescendantsByType(parent, typePath) {
   let tokens = Array.isArray(parent) ? parent : [ parent ];
   for (const type of typePath) {
-    tokens = tokens
-      .flatMap((t) => t.children)
-      .filter((t) => Array.isArray(type) ? type.includes(t.type) : (type === t.type));
+    const predicate = (token) => Array.isArray(type) ? type.includes(token.type) : (type === token.type);
+    tokens = tokens.flatMap((t) => t.children.filter(predicate));
   }
   return tokens;
 }