From: Jonathan Nieder <jrnieder@gmail.com>
Date: Sat, 22 Nov 2014 21:07:25 GMT
Subject: Re: [PATCH 1/6] prune_remote(): exit early if there are no stale references
Message-ID: <20141122210725.GB15320@google.com>
In-Reply-To: <1416578950-23210-2-git-send-email-mhagger@alum.mit.edu>

Michael Haggerty wrote:

> Aside from making the logic clearer, this avoids a call to
> warn_dangling_symrefs(), which always does a for_each_rawref()
> iteration.
>
> Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
> ---
>  builtin/remote.c | 39 +++++++++++++++++++++------------------
>  1 file changed, 21 insertions(+), 18 deletions(-)

I had been wondering about this but didn't chase it down far enough.
Thanks for noticing and cleaning it up.

Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>