Re: [PATCH v2 4/8] refs: factor delete_ref loose ref step into a helper

2013-09-02 Thread Brad King
On 08/31/2013 12:30 PM, Michael Haggerty wrote: > Given that ret is only returned, you could restore the filename before > the if statement and replace the ret variable with an immediate return > statement: Good idea. Fixed in next revision. Thanks, -Brad -- To unsubscribe from this list: send t

Re: [PATCH v2 4/8] refs: factor delete_ref loose ref step into a helper

2013-08-31 Thread Michael Haggerty
On 08/30/2013 08:12 PM, Brad King wrote: > Factor loose ref deletion into helper function delete_ref_loose to allow > later use elsewhere. > > Signed-off-by: Brad King > --- > refs.c | 22 +++--- > 1 file changed, 15 insertions(+), 7 deletions(-) > > diff --git a/refs.c b/refs

[PATCH v2 4/8] refs: factor delete_ref loose ref step into a helper

2013-08-30 Thread Brad King
Factor loose ref deletion into helper function delete_ref_loose to allow later use elsewhere. Signed-off-by: Brad King --- refs.c | 22 +++--- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/refs.c b/refs.c index 2e755b4..5dd86ee 100644 --- a/refs.c +++ b/refs.c