events: fix potential permanent deopt

PR-URL: #13384 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Refael Ackermann <[email protected]>
nodejs · Jun 7, 2017 · a666238 · a666238
1 parent 48cad9c
commit a666238
Showing 1 changed file with 17 additions and 2 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -306,10 +306,25 @@ EventEmitter.prototype.prependListener =
     };
 
 function onceWrapper() {
-  this.target.removeListener(this.type, this.wrapFn);
   if (!this.fired) {
+    this.target.removeListener(this.type, this.wrapFn);
     this.fired = true;
-    this.listener.apply(this.target, arguments);
+    switch (arguments.length) {
+      case 0:
+        return this.listener.call(this.target);
+      case 1:
+        return this.listener.call(this.target, arguments[0]);
+      case 2:
+        return this.listener.call(this.target, arguments[0], arguments[1]);
+      case 3:
+        return this.listener.call(this.target, arguments[0], arguments[1],
+                                  arguments[2]);
+      default:
+        const args = new Array(arguments.length);
+        for (var i = 0; i < args.length; ++i)
+          args[i] = arguments[i];
+        this.listener.apply(this.target, args);
+    }
   }
 }