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

Re: [PATCH v2] refs: speed up is_refname_available

From
Jeff King <peff@peff.net>
Date
Sep 11, 2014, 01:39 UTC
Message-ID
<20140911013903.GA5525@peff.net>
In-Reply-To
<xmqqk35bf764.fsf@gitster.dls.corp.google.com>
On Wed, Sep 10, 2014 at 03:21:39PM -0700, Junio C Hamano wrote:
Show 14 quoted lines
> Jeff King <peff@peff.net> writes:
> 
> > ...
> > Reviewed-by: Michael Haggerty <mhagger@alum.mit.edu>
> > Signed-off-by: Jeff King <peff@peff.net>
> > ---
> > Sorry for the quick v2; Michael and I crossed emails off-list, and I
> > missed some of his review. This version has some minor style and comment
> > fixups.
> 
> Looks sensible from a cursory read, but it conflicts heavily with
> 54e696fc (refs.c: pass a skip list to name_conflict_fn, 2014-07-16)
> which is queued early on 'pu', and I am definitely not in the
> business of re-designing that huge topic myself.

Did you miss the part in v1 where I pointed to a tree with a proposed resolution? :)

I am also happy to rebase if the other bits are coming to master soon.
-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 4 of 7 in “refs: speed up is_refname_available”
  1. refs: speed up is_refname_availableJeff King, Sep 10, 2014
  2. refs: speed up is_refname_availableJeff King, Sep 10, 2014
  3. Junio C HamanoSep 10, 2014
  4. Jeff KingSep 11, 2014
  5. Junio C HamanoSep 10, 2014
  6. Junio C HamanoSep 11, 2014
  7. Jeff KingSep 11, 2014

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.