benchmark: update compare.js exit method

adrian-nitu-92 · AndreasMadsen · commit 4b527a4129c0 · 2016-08-12T13:23:43.000+02:00
Node documentation recommends using process.exitCode = x and returning as a way to exit. PR-URL: #7961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
diff --git a/benchmark/compare.js b/benchmark/compare.js
@@ -33,7 +33,8 @@ const benchmarks = cli.benchmarks();
 
 if (benchmarks.length === 0) {
   console.error('no benchmarks found');
-  process.exit(1);
+  process.exitCode = 1;
+  return;
 }
 
 // Create queue from the benchmarks list such both node versions are tested

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,8 @@ const benchmarks = cli.benchmarks();`
`33`	`33`
`34`	`34`	`if (benchmarks.length === 0) {`
`35`	`35`	`console.error('no benchmarks found');`
`36`		`- process.exit(1);`
	`36`	`+ process.exitCode = 1;`
	`37`	`+ return;`
`37`	`38`	`}`
`38`	`39`
`39`	`40`	`// Create queue from the benchmarks list such both node versions are tested`