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

benchmark: remove unused parameters

Browse Source 
Functions onOnline and onMessage in benchmark/cluster/echo.js
had unused parameters. They were removed.

PR-URL: https://github.com/nodejs/node/pull/14640
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This commit is contained in:
nishijayaraj 2017-08-05 17:37:10 +05:30 committed by James M Snell
parent 1b719fe3d7
commit e67220ec81
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
benchmark/cluster/echo.js
View File

@ -34,7 +34,7 @@ if (cluster.isMaster) {
for (var i = 0; i < workers; ++i) for (var i = 0; i < workers; ++i)
cluster.fork().on('online', onOnline).on('message', onMessage); cluster.fork().on('online', onOnline).on('message', onMessage);
function onOnline(msg) { function onOnline() {
if (++readies === workers) { if (++readies === workers) {
bench.start(); bench.start();
broadcast(); broadcast();
@ -56,7 +56,7 @@ if (cluster.isMaster) {
} }
} }
function onMessage(msg) { function onMessage() {
if (++msgCount === expectedPerBroadcast) { if (++msgCount === expectedPerBroadcast) {
msgCount = 0; msgCount = 0;
broadcast(); broadcast();