Thread (1 message) 1 message, 1 author, 2023-02-08

Re: [PATCH] remote: align columns on -v

From: Junio C Hamano <hidden>
Date: 2023-02-08 16:33:02

Torsten Bögershausen [off-list ref] writes:
Especially, what happens if a remote is named with a non-ASCII
character (unicode code point would be a better term) ?
To determine the width on screen for aligment, strlen()
does the wrong thing here.
Interesting point.  I am still skeptical about the patch and suspect
that it is better to stick to easy-to-parse output, but if we switch
to align for humans, we should definitely count display columns, not
byte count with strlen().

Thanks.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help