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]>

Author: mscdex

lib/net.js

@@ -59,9 +59,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);
@@ -875,9 +874,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);