Conversation
Merging this PR will degrade performance by 8.43%
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: |
|
|
||
| #[cfg(not(windows))] | ||
| { | ||
| unsafe { |
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: |
66afdd0 to
75ae5ea
Compare
|
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
ae1d7de to
5507965
Compare
|
GNU testsuite comparison: |
Summary
--groupingoption to numfmt--to/--format