OSDN Git Service

blame,shortlog: don't make local option variables static
authorJeff King <peff@peff.net>
Mon, 13 Jun 2016 05:39:28 +0000 (01:39 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 13 Jun 2016 17:33:33 +0000 (10:33 -0700)
commit64093fc06a871f71316211a2aea6bb46c49b20ab
tree909e8391c6a00852340a2288dc21e09efd5a60b7
parent7c4b169585ebe650783051c4b7a7b17de62836ad
blame,shortlog: don't make local option variables static

There's no need for these option variables to be static,
except that they are referenced by the options array itself,
which is static. But having all of this static is simply
unnecessary and confusing (and inconsistent with most other
commands, which either use a static global option list or a
true function-local one).

Note that in some cases we may need to actually initialize
the variables (since we cannot rely on BSS to do so). This
is a net improvement to readability, though, as we can use
the more verbose initializers for our string_lists.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/blame.c
builtin/shortlog.c