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

Re: [PATCH] t8002-blame: simplify padding generation in blank boundary tests

From
Junio C Hamano <gitster@pobox.com>
Date
Jan 21, 2025, 20:10 UTC
Message-ID
<xmqqzfjkylky.fsf@gitster.g>
In-Reply-To
<bwo4w2qbbe6czt4n7vshwyn4lbl2ieihqmdugz4fmnhd4qbsd2@tevhcjwigxbi>
Jan Palus <jpalus@fastmail.com> writes:
> Note that original version was "%0.s" in which there is some ambiguity
> whether "0" is a flag or field width and not "%.0s" in which "0" indeed
> would mean precision.

Ah, I missed that part. Also thanks for filling in the "printf from coreutils is the one that has issues with the code".

Show 7 quoted lines
>> Anybody can help that "further polishing as suggested" step, and
>> when the patch is left in limbo for too long, I might step in to do
>> it myself (when I have no other better things to do), but it is
>> customary around here that the original patch submitter does so.
>
> I was about to follow-up but didn't find time. Sorry it took so long.
> I will post v2 shortly.
Thanks.
Previous: Jan Palus
Message 6 of 6 in “t8002-blame: simplify padding generation in blank boundary tests”
  1. t8002-blame: simplify padding generation in blank boundary testsJan Palus, Jan 11, 2025
  2. Patrick SteinhardtJan 13, 2025
  3. Junio C HamanoJan 13, 2025
  4. Junio C HamanoJan 18, 2025
  5. Jan PalusJan 20, 2025
  6. Junio C HamanoJan 21, 2025

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.