Re: [PATCH v11 32/41] refs.c: make delete_ref use a transaction

2014-06-03 Thread Ronnie Sahlberg
Thanks! On Fri, May 30, 2014 at 10:28 AM, Jonathan Nieder wrote: > Ronnie Sahlberg wrote: > >> Signed-off-by: Ronnie Sahlberg >> --- >> refs.c | 34 +- >> 1 file changed, 13 insertions(+), 21 deletions(-) > > Reviewed-by: Jonathan Nieder > > [...] >> +++ b/refs.

Re: [PATCH v11 32/41] refs.c: make delete_ref use a transaction

2014-05-30 Thread Jonathan Nieder
Ronnie Sahlberg wrote: > Signed-off-by: Ronnie Sahlberg > --- > refs.c | 34 +- > 1 file changed, 13 insertions(+), 21 deletions(-) Reviewed-by: Jonathan Nieder [...] > +++ b/refs.c [...] > @@ -2542,24 +2537,21 @@ static int delete_ref_loose(struct ref_lock *lo

[PATCH v11 32/41] refs.c: make delete_ref use a transaction

2014-05-27 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. Signed-off-by: Ronnie Sahlberg --- refs.c | 34 +- 1 file changed, 13 insertions(+), 21 deletio