benchmark: fix CLI arguments check in common.js

vsemozhetbyt · evanlucas · commit 08ba9d437c3b · 2017-05-02T12:46:14.000-05:00
PR-URL: #12429 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/benchmark/common.js b/benchmark/common.js
@@ -43,7 +43,7 @@ Benchmark.prototype._parseArgs = function(argv, configs) {
   // Parse configuration arguments
   for (const arg of argv) {
     const match = arg.match(/^(.+?)=([\s\S]*)$/);
-    if (!match || !match[1]) {
+    if (!match) {
       console.error('bad argument: ' + arg);
       process.exit(1);
     }

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ Benchmark.prototype._parseArgs = function(argv, configs) {`
`43`	`43`	`// Parse configuration arguments`
`44`	`44`	`for (const arg of argv) {`
`45`	`45`	`const match = arg.match(/^(.+?)=([\s\S]*)$/);`
`46`		`- if (!match \|\| !match[1]) {`
	`46`	`+ if (!match) {`
`47`	`47`	`console.error('bad argument: ' + arg);`
`48`	`48`	`process.exit(1);`
`49`	`49`	`}`