Skip to content

numfmt: add --from=auto test for multibyte whitespace separator#13487

Open
kushals256 wants to merge 1 commit into
uutils:mainfrom
kushals256:fix/numfmt-multibyte-whitespace-field
Open

numfmt: add --from=auto test for multibyte whitespace separator#13487
kushals256 wants to merge 1 commit into
uutils:mainfrom
kushals256:fix/numfmt-multibyte-whitespace-field

Conversation

@kushals256

@kushals256 kushals256 commented Jul 21, 2026

Copy link
Copy Markdown

Summary

Rebased onto main after the panic fix landed in 7807c8d (#13482, now closed).

This PR adds pipe-input regression coverage for --from=auto --field=2 when fields are separated by U+3000 IDEOGRAPHIC SPACE.

Test plan

  • cargo test test_field_with_multibyte_whitespace_separator

@sylvestre

Copy link
Copy Markdown
Contributor

please fix the failing jobs

Rebased onto main after 7807c8d landed the panic fix for uutils#13482.
Adds pipe-input coverage for --from=auto with a U+3000 field separator.

Co-authored-by: Cursor <cursoragent@cursor.com>
@kushals256

Copy link
Copy Markdown
Author

Rebased onto main — the panic fix was already merged in 7807c8d (issue #13482 is closed).

This PR now only adds an extra regression test: --from=auto with a U+3000 field separator via pipe input, expecting normalized output 1K 2000.

Conflicts resolved; CI should be green on the next run.

@kushals256
kushals256 force-pushed the fix/numfmt-multibyte-whitespace-field branch from a5143b6 to d9ab981 Compare July 21, 2026 13:54
@kushals256 kushals256 changed the title numfmt: fix panic on multibyte whitespace with --field numfmt: add --from=auto test for multibyte whitespace separator Jul 21, 2026
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.

2 participants