Skip to content

Accept G, GB and GiB suffixes for options#4609

Merged
Cyan4973 merged 2 commits into
facebook:devfrom
Integral-Tech:memory-g-suffix
Mar 13, 2026
Merged

Accept G, GB and GiB suffixes for options#4609
Cyan4973 merged 2 commits into
facebook:devfrom
Integral-Tech:memory-g-suffix

Conversation

@Integral-Tech

@Integral-Tech Integral-Tech commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

Several options currently accept only K, KB, KiB, M, MB and MiB suffixes. This change adds support for G, GB and GiB so users can specify gigabyte values using the same suffix styles.

Closes: #4540

@jlee303 jlee303 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looking good, just a few suggestions, thanks for putting up a pr 😄

Comment thread programs/zstdcli.c
Comment thread tests/cli-tests/basic/memlimit.sh
@Integral-Tech Integral-Tech changed the title Add G, GB and GiB suffix support for --memory option Accept G, GB and GiB suffixes for options Mar 7, 2026
@Integral-Tech
Integral-Tech requested a review from jlee303 March 7, 2026 06:59

@jlee303 jlee303 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm! Left a small suggestion for additional test case :)

Comment thread tests/cli-tests/basic/memlimit.sh
@Integral-Tech
Integral-Tech force-pushed the memory-g-suffix branch 2 times, most recently from 66530ec to f403390 Compare March 10, 2026 02:11
@Integral-Tech

Copy link
Copy Markdown
Contributor Author

@jlee303 Resolved :) Could you please approve the workflows again?

@jlee303

jlee303 commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

@jlee303 Resolved :) Could you please approve the workflows again?

I think it's already been approved by @Cyan4973, I don't see a button on my end :)

@Cyan4973
Cyan4973 merged commit ee0af93 into facebook:dev Mar 13, 2026
105 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The --mamory option should support the G suffix.

3 participants