Re: [PATCH 09/24] refs.c: pass a list of names to skip to is_refname_available

2014-10-03 Thread Junio C Hamano
Jonathan Nieder writes: > Junio C Hamano wrote: > ... >> As cbe73331 (refs: speed up is_refname_available, 2014-09-10) >> touches the same area and is now in 'master', the logic around here >> in this series needs to be reworked. > > Thanks for the heads up. (I hadn't realized the ref-transactio

Re: [PATCH 09/24] refs.c: pass a list of names to skip to is_refname_available

2014-10-03 Thread Jonathan Nieder
Junio C Hamano wrote: > Jonathan Nieder writes: >> diff --git a/refs.c b/refs.c >> index f124c2b..6820c93 100644 >> --- a/refs.c >> +++ b/refs.c >> @@ -801,14 +801,16 @@ static int names_conflict(const char *refname1, const >> char *refname2) >> >> struct name_conflict_cb { >> const char

Re: [PATCH 09/24] refs.c: pass a list of names to skip to is_refname_available

2014-10-02 Thread Junio C Hamano
Jonathan Nieder writes: > diff --git a/refs.c b/refs.c > index f124c2b..6820c93 100644 > --- a/refs.c > +++ b/refs.c > @@ -801,14 +801,16 @@ static int names_conflict(const char *refname1, const > char *refname2) > > struct name_conflict_cb { > const char *refname; > - const char *o

[PATCH 09/24] refs.c: pass a list of names to skip to is_refname_available

2014-10-01 Thread Jonathan Nieder
From: Ronnie Sahlberg Date: Thu, 1 May 2014 11:16:07 -0700 Change is_refname_available to take a list of strings to exclude when checking for conflicts instead of just one single name. We can already exclude a single name for the sake of renames. This generalizes that support. ref_transaction_co