Skip to content

Commit

Permalink
benchmark: fix CLI arguments check in common.js
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
vsemozhetbyt authored and evanlucas committed Apr 25, 2017
1 parent 7ba0234 commit 014d57a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion benchmark/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
Expand Down

0 comments on commit 014d57a

Please sign in to comment.