Skip to content

forbid scientific notation in numfmt #11655#11721

Open
Devel08 wants to merge 2 commits intouutils:mainfrom
Devel08:fix-numfmt-#11655
Open

forbid scientific notation in numfmt #11655#11721
Devel08 wants to merge 2 commits intouutils:mainfrom
Devel08:fix-numfmt-#11655

Conversation

@Devel08
Copy link
Copy Markdown

@Devel08 Devel08 commented Apr 9, 2026

This PR forbids scientific notation in input of numfmt and should resolve #11655. Sorry for opening and closing so much PR's, but I am new in using git with github. That'll be the last one, I swear!
Passes all checks with cargo test .
Thanks!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

GNU testsuite comparison:

GNU test failed: tests/misc/io-errors. tests/misc/io-errors is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/cut/cut-huge-range is now passing!

@cakebaker
Copy link
Copy Markdown
Contributor

Can you please enable the corresponding test in tests/by_util/test_numfmt.rs and run cargo fmt? Thanks.

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.

numfmt: accepts scientific notation (e.g. 1e9) where GNU rejects as invalid suffix

2 participants