benchmark: fix CLI arguments check in common.js · nodejs/node@014d57a · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 37.5k
• Star 122k

• Code
• Issues 615
• Pull requests 564
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 014d57a

Browse filesBrowse the repository at this point in the historyBrowse files

vsemozhetbyt

authored and

evanlucas

committed

benchmark: fix CLI arguments check in common.js

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>

1 parent 7ba0234 commit 014d57a
Copy full SHA for 014d57a

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• benchmark
  • common.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎benchmark/common.js‎

Copy file name to clipboardExpand all lines: benchmark/common.js

+1-1Lines changed: 1 addition & 1 deletion

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	}

0 commit comments

Comments

0 (0)