events: optimize arrayClone by copying forward

Optimize arrayClone by copying forward. It's slightly faster (and more readable) to copy array elements in forward direction. This way it also avoids the ToBoolean and the postfix count operation. PR-URL: #10571 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]>
nodejs · Jan 4, 2017 · f2f997a · f2f997a
1 parent 4198253
commit f2f997a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -477,9 +477,9 @@ function spliceOne(list, index) {
   list.pop();
 }
 
-function arrayClone(arr, i) {
-  var copy = new Array(i);
-  while (i--)
+function arrayClone(arr, n) {
+  var copy = new Array(n);
+  for (var i = 0; i < n; ++i)
     copy[i] = arr[i];
   return copy;
 }