lib: add bound apply variants of varargs primordials

Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
nodejs · Jan 20, 2021 · 60dab86 · 60dab86
1 parent e3e054d
commit 60dab86
Show file tree

Hide file tree

Showing 4 changed files with 55 additions and 12 deletions.
diff --git a/lib/internal/per_context/primordials.js b/lib/internal/per_context/primordials.js
@@ -21,10 +21,28 @@ const {
 // `uncurryThis` is equivalent to `func => Function.prototype.call.bind(func)`.
 // It is using `bind.bind(call)` to avoid using `Function.prototype.bind`
 // and `Function.prototype.call` after it may have been mutated by users.
-const { bind, call } = Function.prototype;
+const { apply, bind, call } = Function.prototype;
 const uncurryThis = bind.bind(call);
 primordials.uncurryThis = uncurryThis;
 
+const applyBind = bind.bind(apply);
+primordials.applyBind = applyBind;
+
+// Methods that accept a variable number of arguments, and thus it's useful to
+// also create `${prefix}${key}Apply`, which uses `Function.prototype.apply`,
+// instead of `Function.prototype.call`, and thus doesn't require iterator
+// destructuring.
+const varargsMethods = [
+  'ArrayPrototypePush',
+  'ArrayPrototypeUnshift',
+  'ArrayOf',
+  'MathMax',
+  'MathMin',
+  'MathHypot',
+  'StringPrototypeConcat',
+  'TypedArrayOf',
+];
+
 function getNewKey(key) {
   return typeof key === 'symbol' ?
     `Symbol${key.description[7].toUpperCase()}${key.description.slice(8)}` :
@@ -51,7 +69,13 @@ function copyPropsRenamed(src, dest, prefix) {
     if ('get' in desc) {
       copyAccessor(dest, prefix, newKey, desc);
     } else {
-      ReflectDefineProperty(dest, `${prefix}${newKey}`, desc);
+      const name = `${prefix}${newKey}`;
+      ReflectDefineProperty(dest, name, desc);
+      if (varargsMethods.includes(name)) {
+        ReflectDefineProperty(dest, `${name}Apply`, {
+          value: applyBind(desc.value, src),
+        });
+      }
     }
   }
 }
@@ -63,10 +87,18 @@ function copyPropsRenamedBound(src, dest, prefix) {
     if ('get' in desc) {
       copyAccessor(dest, prefix, newKey, desc);
     } else {
-      if (typeof desc.value === 'function') {
-        desc.value = desc.value.bind(src);
+      const { value } = desc;
+      if (typeof value === 'function') {
+        desc.value = value.bind(src);
+      }
+
+      const name = `${prefix}${newKey}`;
+      ReflectDefineProperty(dest, name, desc);
+      if (varargsMethods.includes(name)) {
+        ReflectDefineProperty(dest, `${name}Apply`, {
+          value: applyBind(value, src),
+        });
       }
-      ReflectDefineProperty(dest, `${prefix}${newKey}`, desc);
     }
   }
 }
@@ -78,10 +110,18 @@ function copyPrototype(src, dest, prefix) {
     if ('get' in desc) {
       copyAccessor(dest, prefix, newKey, desc);
     } else {
-      if (typeof desc.value === 'function') {
-        desc.value = uncurryThis(desc.value);
+      const { value } = desc;
+      if (typeof value === 'function') {
+        desc.value = uncurryThis(value);
+      }
+
+      const name = `${prefix}${newKey}`;
+      ReflectDefineProperty(dest, name, desc);
+      if (varargsMethods.includes(name)) {
+        ReflectDefineProperty(dest, `${name}Apply`, {
+          value: applyBind(value, src),
+        });
       }
-      ReflectDefineProperty(dest, `${prefix}${newKey}`, desc);
     }
   }
 }

diff --git a/lib/internal/process/per_thread.js b/lib/internal/process/per_thread.js
@@ -8,6 +8,7 @@ const {
   ArrayIsArray,
   ArrayPrototypeMap,
   ArrayPrototypePush,
+  ArrayPrototypePushApply,
   ArrayPrototypeSplice,
   BigUint64Array,
   Float64Array,
@@ -274,7 +275,7 @@ function buildAllowedFlags() {
       if (recursiveExpansion) {
         if (recursiveExpansion[0] === to)
           ArrayPrototypeSplice(recursiveExpansion, 0, 1);
-        ArrayPrototypePush(expansion, ...recursiveExpansion);
+        ArrayPrototypePushApply(expansion, recursiveExpansion);
         continue;
       }
       isAccepted = options.get(to).envVarSettings === kAllowedInEnvironment;

diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -6,6 +6,7 @@ const {
   ArrayPrototypeFilter,
   ArrayPrototypeForEach,
   ArrayPrototypePush,
+  ArrayPrototypePushApply,
   ArrayPrototypeSort,
   ArrayPrototypeUnshift,
   BigIntPrototypeValueOf,
@@ -664,7 +665,7 @@ function getKeys(value, showHidden) {
   if (showHidden) {
     keys = ObjectGetOwnPropertyNames(value);
     if (symbols.length !== 0)
-      ArrayPrototypePush(keys, ...symbols);
+      ArrayPrototypePushApply(keys, symbols);
   } else {
     // This might throw if `value` is a Module Namespace Object from an
     // unevaluated module, but we don't want to perform the actual type
@@ -680,7 +681,7 @@ function getKeys(value, showHidden) {
     }
     if (symbols.length !== 0) {
       const filter = (key) => ObjectPrototypePropertyIsEnumerable(value, key);
-      ArrayPrototypePush(keys, ...ArrayPrototypeFilter(symbols, filter));
+      ArrayPrototypePushApply(keys, ArrayPrototypeFilter(symbols, filter));
     }
   }
   return keys;

diff --git a/lib/repl.js b/lib/repl.js
@@ -52,6 +52,7 @@ const {
   ArrayPrototypeMap,
   ArrayPrototypePop,
   ArrayPrototypePush,
+  ArrayPrototypePushApply,
   ArrayPrototypeReverse,
   ArrayPrototypeShift,
   ArrayPrototypeSort,
@@ -1310,7 +1311,7 @@ function complete(line, callback) {
       if (!this.useGlobal) {
         // When the context is not `global`, builtins are not own
         // properties of it.
-        ArrayPrototypePush(contextOwnNames, ...globalBuiltins);
+        ArrayPrototypePushApply(contextOwnNames, globalBuiltins);
       }
       ArrayPrototypePush(completionGroups, contextOwnNames);
       if (filter !== '') addCommonWords(completionGroups);