chore: use .concat for better perfs (1 less creation of arrays)

gvergnaud · Aug 8, 2024 · 7f2aaf3 · 7f2aaf3
1 parent 423b75b
commit 7f2aaf3
Showing 1 changed file with 16 additions and 17 deletions.
diff --git a/src/internals/helpers.ts b/src/internals/helpers.ts
@@ -103,23 +103,22 @@ export const matchPattern = (
         : false;
     }
 
-    return [
-      ...Object.getOwnPropertySymbols(pattern),
-      ...Object.keys(pattern),
-    ].every((k: string | symbol): boolean => {
-      // @ts-ignore
-      const subPattern = pattern[k];
-
-      return (
-        (k in value || isOptionalPattern(subPattern)) &&
-        matchPattern(
-          subPattern,
-          // @ts-ignore
-          value[k],
-          select
-        )
-      );
-    });
+    return (Object.keys(pattern) as Array<string | symbol>)
+      .concat(Object.getOwnPropertySymbols(pattern))
+      .every((k: string | symbol): boolean => {
+        // @ts-ignore
+        const subPattern = pattern[k];
+
+        return (
+          (k in value || isOptionalPattern(subPattern)) &&
+          matchPattern(
+            subPattern,
+            // @ts-ignore
+            value[k],
+            select
+          )
+        );
+      });
   }
 
   return Object.is(value, pattern);