Skip to content

Commit

Permalink
benchmark: remove unused parameters
Browse files Browse the repository at this point in the history
Functions onOnline and onMessage in benchmark/cluster/echo.js
had unused parameters. They were removed.

PR-URL: #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>
  • Loading branch information
nishijayaraj authored and addaleax committed Aug 10, 2017
1 parent 935d34b commit 574cc37
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions benchmark/cluster/echo.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ if (cluster.isMaster) {
for (var i = 0; i < workers; ++i)
cluster.fork().on('online', onOnline).on('message', onMessage);

function onOnline(msg) {
function onOnline() {
if (++readies === workers) {
bench.start();
broadcast();
Expand All @@ -56,7 +56,7 @@ if (cluster.isMaster) {
}
}

function onMessage(msg) {
function onMessage() {
if (++msgCount === expectedPerBroadcast) {
msgCount = 0;
broadcast();
Expand Down

0 comments on commit 574cc37

Please sign in to comment.