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

Re: What's cooking (draft for #4's issue this month)

From
Junio C Hamano <gitster@pobox.com>
Date
Apr 14, 2021, 21:53 UTC
Message-ID
<xmqqv98own1s.fsf@gitster.g>
In-Reply-To
<xmqqeefcy35i.fsf@gitster.g>
Junio C Hamano <gitster@pobox.com> writes:
>> The following patch fixes this, which
>> I'll include in my next reroll of this series.

Hmph, I am not sure if that is wise. The offending "-1 no longer is accepted" comes from a topic that is not even in 'next', so you may not want to depend on it.

It turns out that the two-argument form that passes -1 to the function we see below as the preimage in your fix-up patch comes from my conflict resolution. Your original has

	type_from_string_gently(v0, -1, 1);

and the other topic wants to make two unrelated changes to the API, i.e. making -1 no longer a valid "please count, as it is pointless to force callers to always count" option, and drops "is this asking to be gentle?" parameter. It may be better to just update what is recorded in my conflict resolution machinery, without making it

	type_from_string_gently(v0, strlen(v0), 1);

as it would be necessary to adjust when both topics are merged anyway.

Previous: Junio C HamanoNext: Jeff King
Message 4 of 14 in “What's cooking (draft for #4's issue this month)”
  1. Junio C HamanoApr 14, 2021
  2. Patrick SteinhardtApr 14, 2021
  3. Junio C HamanoApr 14, 2021
  4. Junio C HamanoApr 14, 2021
  5. Jeff KingApr 15, 2021
  6. Junio C HamanoApr 15, 2021
  7. Junio C HamanoApr 14, 2021
  8. Junio C HamanoApr 14, 2021
  9. Junio C HamanoApr 14, 2021
  10. brian m. carlsonApr 15, 2021
  11. Junio C HamanoApr 15, 2021
  12. brian m. carlsonApr 19, 2021
  13. Junio C HamanoApr 19, 2021
  14. Han-Wen NienhuysApr 15, 2021

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.