Re: [PATCH v20 39/48] refs.c: make delete_ref use a transaction

2014-07-14 Thread Ronnie Sahlberg
On Tue, Jul 8, 2014 at 6:52 AM, Michael Haggerty wrote: > On 06/20/2014 04:43 PM, Ronnie Sahlberg wrote: >> Change delete_ref to use a ref transaction for the deletion. At the same time >> since we no longer have any callers of repack_without_ref we can now delete >> this function. >> >> Change de

Re: [PATCH v20 39/48] refs.c: make delete_ref use a transaction

2014-07-08 Thread Michael Haggerty
On 06/20/2014 04:43 PM, Ronnie Sahlberg wrote: > Change delete_ref to use a ref transaction for the deletion. At the same time > since we no longer have any callers of repack_without_ref we can now delete > this function. > > Change delete_ref to return 0 on success and 1 on failure instead of the

[PATCH v20 39/48] refs.c: make delete_ref use a transaction

2014-06-20 Thread Ronnie Sahlberg
Change delete_ref to use a ref transaction for the deletion. At the same time since we no longer have any callers of repack_without_ref we can now delete this function. Change delete_ref to return 0 on success and 1 on failure instead of the previous 0 on success either 1 or -1 on failure. Review