Re: [PATCH v8 34/44] refs.c: make prune_ref use a transaction to delete the ref

2014-05-22 Thread Ronnie Sahlberg
Added a comment that any flags =0x100 are reserved for internal use. On Wed, May 21, 2014 at 4:01 PM, Jonathan Nieder jrnie...@gmail.com wrote: Ronnie Sahlberg wrote: Change prune_ref to delete the ref using a ref transaction. To do this we also need to add a new flag REF_ISPRUNING that

Re: [PATCH v8 34/44] refs.c: make prune_ref use a transaction to delete the ref

2014-05-21 Thread Jonathan Nieder
Ronnie Sahlberg wrote: Change prune_ref to delete the ref using a ref transaction. To do this we also need to add a new flag REF_ISPRUNING that will tell the transaction that we do not want to delete this ref from the packed refs. Interesting. Since the flag is per ref update, it even would

[PATCH v8 34/44] refs.c: make prune_ref use a transaction to delete the ref

2014-05-15 Thread Ronnie Sahlberg
Change prune_ref to delete the ref using a ref transaction. To do this we also need to add a new flag REF_ISPRUNING that will tell the transaction that we do not want to delete this ref from the packed refs. This flag is private to refs.c and not exposed to external callers. Signed-off-by: Ronnie