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
Jeff King <peff@peff.net>
Date
Apr 15, 2021, 09:37 UTC
Message-ID
<YHgJdtF4X2LQCuZ0@coredump.intra.peff.net>
In-Reply-To
<xmqqv98own1s.fsf@gitster.g>
On Wed, Apr 14, 2021 at 02:53:19PM -0700, Junio C Hamano wrote:
Show 25 quoted lines
> 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.
Heh. I wonder if I was too optimistic in my review at:
  https://lore.kernel.org/git/YHCYfeRsHU34ZF%2Fl@coredump.intra.peff.net/
-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 5 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.