From: Felipe Contreras Date: Sun, 07 Apr 2013 07:18:59 GMT Subject: [PATCH] format-patch: trivial cleanup Message-ID: <1365319139-12362-1-git-send-email-felipe.contreras@gmail.com> Signed-off-by: Felipe Contreras --- Actually, now that we are on it. After applying my latest patch series, this makes sense. builtin/log.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/builtin/log.c b/builtin/log.c index fab7998..7cf9277 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -1058,12 +1058,10 @@ static char *find_branch_name(struct rev_info *rev) else return NULL; } - if (0 <= positive) { - ref = rev->cmdline.rev[positive].name; - tip_sha1 = rev->cmdline.rev[positive].item->sha1; - } else { + if (positive < 0) return NULL; - } + ref = rev->cmdline.rev[positive].name; + tip_sha1 = rev->cmdline.rev[positive].item->sha1; if (dwim_ref(ref, strlen(ref), branch_sha1, &full_ref) && !prefixcmp(full_ref, "refs/heads/") && !hashcmp(tip_sha1, branch_sha1)) -- 1.8.2