events: improve removeListener() performance

array.shift() seems to be faster than arrayClone() when the item to remove is at the front (at least with V8 5.4). PR-URL: #10572 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Jan 27, 2017 · ded1757 · ded1757
1 parent 2f4577c
commit ded1757
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -360,6 +360,8 @@ EventEmitter.prototype.removeListener =
           } else {
             delete events[type];
           }
+        } else if (position === 0) {
+          list.shift();
         } else {
           spliceOne(list, position);
         }