Skip to content

bugfix quiet option : no argument needed#16

Merged
jnavila merged 1 commit intojnavila:suppress_output_optionfrom
brockauly:suppress_output_option
Jun 19, 2025
Merged

bugfix quiet option : no argument needed#16
jnavila merged 1 commit intojnavila:suppress_output_optionfrom
brockauly:suppress_output_option

Conversation

@brockauly
Copy link
Copy Markdown

With the previous optstring, the -q option requires an argument and thus takes over the next argument (which is logically the memsize).

Thanks for this branch btw, very helpful

@lucaceresoli
Copy link
Copy Markdown
Contributor

Unless the policy has changed, this repository is not meant for memtester development, it's just a git-managed copy of the original code.

However this is a trivial fix, and looks correct, so FWIW:
Reviewed-by: Luca Ceresoli [email protected]

@jnavila jnavila merged commit bc57f08 into jnavila:suppress_output_option Jun 19, 2025
@jnavila
Copy link
Copy Markdown
Owner

jnavila commented Jun 19, 2025

Thank you!

As @lucaceresoli this is only supposed to be a light fork with custom changes that I'm too lazy to push upstream.

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.

3 participants