Re: Truncating file names with Unicode characters
- Date: Sat, 9 Jun 2018 02:23:51 -0400
- From: Jeff King <peff@xxxxxxxx>
- Subject: Re: Truncating file names with Unicode characters
On Fri, Jun 08, 2018 at 09:25:29PM +0300, Vitaly Potyarkin wrote:
> # Truncating file names with Unicode characters
> When shortening file names that contain Unicode characters, git performs
> truncation without awareness of two-byte characters. That often leads to
> splitting a character in half and displaying a garbage byte that's left.
> Unawareness of Unicode also means that filename length is calculated incorrectly
> and some output gets misaligned.
Yeah, I agree it would be nice for this to be more aware of multi-byte
Unfortunately, we don't actually know what encoding the paths are in
(and they might not even all be in the same encoding). But hopefully we
could at least make the common case work a bit better. If we know that
core.quotepath is off, then the user is expecting paths in their
terminal encoding. So possibly we could just rely on
i18n.logOutputEncoding at that point (which of course defaults to utf8).
If anyone is interested in working on it, the relevant code is in
diff.c:show_stats(). Looks like we do a straight strlen() to get the
width, which should become utf8_strwidth(). That would solve the
problem where we use the wrong width.
To deal with the broken code points, I think you'd need to copy the name
into a strbuf and use strbuf_utf8_replace() to truncate (it doesn't seem
to be well documented, but you can grep for a few other callers).