From: Jonathan Nieder Date: Tue, 15 Jul 2014 23:20:51 GMT Subject: Re: [PATCH] refs.c: add a public is_branch function Message-ID: <20140715232051.GL12427@google.com> In-Reply-To: <1405465358-27054-2-git-send-email-sahlberg@google.com> Ronnie Sahlberg wrote: > Signed-off-by: Ronnie Sahlberg > --- > builtin/fsck.c | 5 ----- > refs.c | 2 +- > refs.h | 2 ++ > 3 files changed, 3 insertions(+), 6 deletions(-) Makes sense -- thanks. (This is an old one: v1.5.4-rc4~27 (2008-01-15), v1.5.4-rc4~30 (2008-01-15). Most of the running time of fsck is per-object, not per-ref, so maintainability here seems worth the performance cost.) Reviewed-by: Jonathan Nieder