Conversation
Merging this PR will degrade performance by 9.11%
Performance Changes
Comparing Footnotes
|
|
GNU testsuite comparison: |
1 similar comment
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
1 similar comment
|
GNU testsuite comparison: |
src/uu/numfmt/src/format.rs
Outdated
There was a problem hiding this comment.
the unsafe is way too big
please give your to AI the guidance to avoid duplication of code ;)
we probably have what we need in src/uucore/src/lib/features/i18n/decimal.rs already
|
please tell your AI to avoid duplication of code and make it smaller |
|
Merge issue |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
sorry - many jobs have been failing |
|
GNU testsuite comparison: |
- add --grouping CLI and validation\n- apply locale decimal/grouping separators in formatting\n- support locale-aware parsing without libc localeconv\n- keep max-whitespace behavior and related tests
66afdd0 to
75ae5ea
Compare
|
GNU testsuite comparison: |
Summary
--groupingoption to numfmt--to/--format