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: nodejs#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 jasnell committed Apr 17, 2017
1 parent 42d0f8b commit e34f8e1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions benchmark/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ Benchmark.prototype._parseArgs = function(argv, configs) {
const extraOptions = {};
// Parse configuration arguments
for (const arg of argv) {
const match = arg.match(/^(.+?)=([\s\S]*)$/);
if (!match || !match[1]) {
const match = arg.match(/^(.+?)=([\s\S]+)$/);
if (!match) {
console.error('bad argument: ' + arg);
process.exit(1);
}
Expand Down

0 comments on commit e34f8e1

Please sign in to comment.