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

cluster: replace forEach with for-of loop

Browse Source 
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
PR-URL: https://github.com/nodejs/node/pull/50317
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
This commit is contained in:
Jérôme Benoit 2024-05-12 19:58:40 +02:00 committed by GitHub
parent c8805b8043
commit 7b2dc79437
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/internal/cluster/child.js
View File

@ -270,14 +270,14 @@ function _disconnect(primaryInitiated) {
}
}
handles.forEach((handle) => {
for (const handle of handles.values()) {
waitingCount++;
if (handle[owner_symbol])
handle[owner_symbol].close(checkWaitingCount);
else
handle.close(checkWaitingCount);
});
}
handles.clear();
checkWaitingCount();

4
lib/internal/cluster/primary.js
View File

@ -152,10 +152,10 @@ function removeWorker(worker) {
function removeHandlesForWorker(worker) {
assert(worker);
handles.forEach((handle, key) => {
for (const { 0: key, 1: handle } of handles) {
if (handle.remove(worker))
handles.delete(key);
});
}
}
cluster.fork = function(env) {