git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 8, 2023, 16:32 UTC
Message-ID
<xmqq1qn0ungo.fsf@gitster.g>
In-Reply-To
<20230208161222.guqqwlceqiy3jq7e@tb-raspi4>
Torsten Bögershausen <tboegi@web.de> 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.
Previous: Torsten BögershausenNext: Kirill Kolyshkin
Message 3 of 7 in “remote: align columns on -v”
  1. remote: align columns on -vKir Kolyshkin, Feb 7, 2023
  2. Torsten BögershausenFeb 8, 2023
  3. Junio C HamanoFeb 8, 2023
  4. Kirill KolyshkinFeb 9, 2023
  5. Junio C HamanoFeb 8, 2023
  6. Kirill KolyshkinFeb 9, 2023
  7. remote: align columns on -vKir Kolyshkin, Feb 9, 2023

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.