Skip to content

Commit

Permalink
benchmark: remove unused parameters
Browse files Browse the repository at this point in the history
Functions completeConfig, completeRun, and
updateProgress had unused parameters. These
were removed.

PR-URL: #14526
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
mnalsup authored and addaleax committed Aug 1, 2017
1 parent 37e55bf commit 0cf1e22
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions benchmark/_benchmark_progress.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,12 +65,12 @@ class BenchmarkProgress {
this.updateProgress();
}

completeConfig(data) {
completeConfig() {
this.completedConfig++;
this.updateProgress();
}

completeRun(job) {
completeRun() {
this.completedRuns++;
this.updateProgress();
}
Expand Down Expand Up @@ -108,7 +108,7 @@ class BenchmarkProgress {
`${caption} `;
}

updateProgress(finished) {
updateProgress() {
if (!process.stderr.isTTY || process.stdout.isTTY) {
return;
}
Expand Down

0 comments on commit 0cf1e22

Please sign in to comment.