benchmark: remove unreachable return

PR-URL: https://github.com/nodejs/node/pull/25883
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>

benchmark/_cli.js

@@ -70,7 +70,6 @@ function CLI(usage, settings) {
     } else {
       // Bad case, abort
       this.abort(usage);
-      return;
     }
   }
 }

benchmark/common.js

@@ -169,7 +169,6 @@ Benchmark.prototype._run = function() {
     child.on('close', (code) => {
       if (code) {
         process.exit(code);
-        return;
       }
 
       if (queueIndex + 1 < self.queue.length) {

benchmark/compare.js

@@ -25,7 +25,6 @@ const cli = CLI(`usage: ./node compare.js [options] [--] <category> ...
 
 if (!cli.optional.new || !cli.optional.old) {
   cli.abort(cli.usage);
-  return;
 }
 
 const binaries = ['old', 'new'];
@@ -98,7 +97,6 @@ if (showProgress) {
   child.once('close', (code) => {
     if (code) {
       process.exit(code);
-      return;
     }
     if (showProgress) {
       progress.completeRun(job);

benchmark/run.js

@@ -67,7 +67,6 @@ if (format === 'csv') {
   child.once('close', (code) => {
     if (code) {
       process.exit(code);
-      return;
     }
 
     // If there are more benchmarks execute the next

benchmark/scatter.js

@@ -17,7 +17,6 @@ const cli = CLI(`usage: ./node scatter.js [options] [--] <filename>
 
 if (cli.items.length !== 1) {
   cli.abort(cli.usage);
-  return;
 }
 
 // Create queue from the benchmarks list such both node versions are tested