GNU Core Utilities - Bugs: bug #29138, expand doesn't support multibyte...
You are not allowed to post comments on this tracker with your current authentication level.
bug #29138: expand doesn't support multibyte characters (e.g. UTF-8) (patch available)
Submitter: | Herbert Thielen <hthielen> | ||
Submitted: | Tue 09 Mar 2010 02:56:28 PM UTC | ||
Category: | None | Severity: | 3 - Normal |
Item Group: | None | Status: | Confirmed |
Privacy: | Public | Assigned to: | pixelbeat |
Open/Closed: | Open |
Tue 09 Mar 2010 06:18:09 PM UTC, comment #2: |
Herbert Thielen <hthielen> |
Tue 09 Mar 2010 04:10:21 PM UTC, comment #1: Thanks for the patch!
|
Pádraig Brady <pixelbeat>![]() ![]() |
Tue 09 Mar 2010 02:56:28 PM UTC, original submission:
'expand' doesn't replace tab characters by the correct number of spaces in case the input data contains multibyte characters, e.g. german umlauts when using the locale UTF-8.
|
Herbert Thielen <hthielen> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 7 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-03-09 | hthielen | Attached File | - | ![]() |
Added expand.c.patch, #19895 |
2010-03-09 | pixelbeat | Attached File | - | ![]() |
Added expand-i18n.diff, #19894 |
2010-03-09 | pixelbeat | Status | None | ![]() |
Confirmed |
Assigned to | None | ![]() |
pixelbeat | ||
2010-03-09 | hthielen | Attached File | - | ![]() |
Added expand-wrong-with-utf8-test.input, #19891 |
Attached File | - | ![]() |
Added expand-wrong-with-utf8-test.expected-output, #19892 | ||
Attached File | - | ![]() |
Added expand.c.patch, #19893 |
Thanks for the hint. As I noted, I'm not experienced with wide characters ... so I didn't realize that unicode may contain combined characters. I've added a wcwidth() call now to forward the column, so it should handle combining characters and multi column characters correctly.
The modified patch is attached.
(file #19895)