lib: remove Reflect.apply where appropriate

Using Reflect.apply where the callback context does not need
to change is unnecessary and less performant.

PR-URL: https://github.com/nodejs/node/pull/27349
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/lib/internal/timers.js b/lib/internal/timers.js
index 566d7df036..8ba7923e57 100644
--- a/lib/internal/timers.js
+++ b/lib/internal/timers.js
@@ -72,7 +72,7 @@
 // timers within (or creation of a new list). However, these operations combined
 // have shown to be trivial in comparison to other timers architectures.
 
-const { Math, Object, Reflect } = primordials;
+const { Math, Object } = primordials;
 
 const {
   scheduleTimer,
@@ -438,7 +438,7 @@ function getTimerCallbacks(runNextTicks) {
         if (!argv)
           immediate._onImmediate();
         else
-          Reflect.apply(immediate._onImmediate, immediate, argv);
+          immediate._onImmediate(...argv);
       } finally {
         immediate._onImmediate = null;
 
@@ -530,7 +530,7 @@ function getTimerCallbacks(runNextTicks) {
         if (args === undefined)
           timer._onTimeout();
         else
-          Reflect.apply(timer._onTimeout, timer, args);
+          timer._onTimeout(...args);
       } finally {
         if (timer._repeat && timer._idleTimeout !== -1) {
           timer._idleTimeout = timer._repeat;