net: remove unnecessary variables

V8 is smart enough to optimize the length property checking when iterating over an array with a for loop. PR-URL: #8112 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Aug 24, 2016 · 17b8381 · 17b8381
1 parent d9142b4
commit 17b8381
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/lib/net.js b/lib/net.js
@@ -60,9 +60,8 @@ exports.createServer = function(options, connectionListener) {
 // connect(path, [cb]);
 //
 exports.connect = exports.createConnection = function() {
-  const argsLen = arguments.length;
-  var args = new Array(argsLen);
-  for (var i = 0; i < argsLen; i++)
+  var args = new Array(arguments.length);
+  for (var i = 0; i < arguments.length; i++)
     args[i] = arguments[i];
   args = normalizeConnectArgs(args);
   debug('createConnection', args);
@@ -885,9 +884,8 @@ Socket.prototype.connect = function(options, cb) {
     // Old API:
     // connect(port, [host], [cb])
     // connect(path, [cb]);
-    const argsLen = arguments.length;
-    var args = new Array(argsLen);
-    for (var i = 0; i < argsLen; i++)
+    var args = new Array(arguments.length);
+    for (var i = 0; i < arguments.length; i++)
       args[i] = arguments[i];
     args = normalizeConnectArgs(args);
     return Socket.prototype.connect.apply(this, args);