Skip to content

Fix argument count check to reflect command line help#103

Merged
smarr merged 2 commits intosmarr:masterfrom
ceresek:fix/harness-min-arg-count
Mar 22, 2026
Merged

Fix argument count check to reflect command line help#103
smarr merged 2 commits intosmarr:masterfrom
ceresek:fix/harness-min-arg-count

Conversation

@ceresek
Copy link
Copy Markdown
Contributor

@ceresek ceresek commented Mar 14, 2026

Java harness help says invocation with single argument (benchmark name) is possible, using default outer and inner iteration counts, but the argument check wanted two arguments minimum.

Copy link
Copy Markdown
Owner

@smarr smarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, that's probably a bug porting from another language.
They all seem to differ in what goes into that array...

Thank you!

Comment thread benchmarks/Java/src/Harness.java Outdated
Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr merged commit 74306fe into smarr:master Mar 22, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants