[PATCH 2/2] branch: clean up repeated strlen
- From
Jeff King <peff@peff.net>
- Date
- Feb 19, 2009, 03:35 UTC
- Message-ID
- <20090219033545.GB18970@coredump.intra.peff.net>
- In-Reply-To
- <20090219033329.GA13666@coredump.intra.peff.net>
Commit 45e2b61 fixed the initialization of a "len" struct parameter via strlen. We can use that to clean up what is now 3 strlens in a 6-line sequence.
Signed-off-by: Jeff King <peff@peff.net> --- I guess a good compiler could optimize these out, but I think it actually reads a little bit nicer.
builtin-branch.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/builtin-branch.c b/builtin-branch.c index 13e4de8..14d4b91 100644 --- a/builtin-branch.c +++ b/builtin-branch.c @@ -443,8 +443,8 @@ static void print_ref_list(int kinds, int detached, int verbose, int abbrev, str item.kind = REF_LOCAL_BRANCH; item.dest = NULL; item.commit = head_commit; - if (strlen(item.name) > ref_list.maxwidth) - ref_list.maxwidth = strlen(item.name); + if (item.len > ref_list.maxwidth) + ref_list.maxwidth = item.len; print_ref_item(&item, ref_list.maxwidth, verbose, abbrev, 1, ""); free(item.name); }
-- 1.6.2.rc1.210.g1210c