1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

net: avoid using forEach

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/11582
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
James M Snell 2017-02-26 18:08:04 -08:00
parent 4d090855c6
commit 42e55eb37b
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
lib/net.js
View File

@ -1524,9 +1524,9 @@ Server.prototype.getConnections = function(cb) {
if (--left === 0) return end(null, total); if (--left === 0) return end(null, total);
} }
this._slaves.forEach(function(slave) { for (var n = 0; n < this._slaves.length; n++) {
slave.getConnections(oncount); this._slaves[n].getConnections(oncount);
}); }
}; };
@ -1562,9 +1562,8 @@ Server.prototype.close = function(cb) {
this._connections++; this._connections++;
// Poll slaves // Poll slaves
this._slaves.forEach(function(slave) { for (var n = 0; n < this._slaves.length; n++)
slave.close(onSlaveClose); this._slaves[n].close(onSlaveClose);
});
} else { } else {
this._emitCloseIfDrained(); this._emitCloseIfDrained();
} }