Re: [PATCH] refs: speed up is_refname_available

2014-09-11 Thread Jeff King
On Thu, Sep 11, 2014 at 10:07:28AM -0700, Junio C Hamano wrote: > Jeff King writes: > > > This has a fairly straightforward conflict with the ref-transaction > > stuff in pu. The "oldrefname" parameter to is_refname_available became a > > list of items; > > Hmph, the trouble I had while reading

Re: [PATCH] refs: speed up is_refname_available

2014-09-11 Thread Junio C Hamano
Jeff King writes: > This has a fairly straightforward conflict with the ref-transaction > stuff in pu. The "oldrefname" parameter to is_refname_available became a > list of items; Hmph, the trouble I had while reading the conflicts was about the new "we skip these when repacking", not "oldrefnam

[PATCH] refs: speed up is_refname_available

2014-09-10 Thread Jeff King
Our filesystem ref storage does not allow D/F conflicts; so if "refs/heads/a/b" exists, we do not allow "refs/heads/a" to exist (and vice versa). This falls out naturally for loose refs, where the filesystem enforces the condition. But for packed-refs, we have to make the check ourselves. We do so