lib: refactor primordials.uncurryThis

aduh95 · targos · commit bb4900d9ebf3 · 2021-06-11T09:29:16.000+02:00
This is done to avoid creating an array and gain performance. Co-authored-by: ExE Boss <3889017+ExE-Boss@users.noreply.github.com> PR-URL: #36221 Refs: https://v8.dev/blog/v8-release-80#optimizing-higher-order-builtins Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
diff --git a/lib/internal/per_context/primordials.js b/lib/internal/per_context/primordials.js
@@ -12,17 +12,11 @@
 // `primordials.Object` where `primordials` is a lexical variable passed
 // by the native module compiler.
 
-const ReflectApply = Reflect.apply;
-
-// This function is borrowed from the function with the same name on V8 Extras'
-// `utils` object. V8 implements Reflect.apply very efficiently in conjunction
-// with the spread syntax, such that no additional special case is needed for
-// function calls w/o arguments.
-// Refs: https://github.com/v8/v8/blob/d6ead37d265d7215cf9c5f768f279e21bd170212/src/js/prologue.js#L152-L156
-function uncurryThis(func) {
-  return (thisArg, ...args) => ReflectApply(func, thisArg, args);
-}
-
+// `uncurryThis` is equivalent to `func => Function.prototype.call.bind(func)`.
+// It is using `call.bind(bind, call)` to avoid using `Function.prototype.bind`
+// after it may have been mutated by users.
+const { bind, call } = Function.prototype;
+const uncurryThis = call.bind(bind, call);
 primordials.uncurryThis = uncurryThis;
 
 function copyProps(src, dest) {
