diff options
author | 2013-10-24 04:45:13 -0400 | |
---|---|---|
committer | 2013-10-24 15:41:56 -0700 | |
commit | a4165851e769ebc32da48904d58f144f2d93e69a (patch) | |
tree | fba46378665b3d9c8d8495920e085a06376e0a3c /Documentation/RelNotes/1.6.6.txt | |
parent | make get_short_ref a public function (diff) | |
download | tgif-a4165851e769ebc32da48904d58f144f2d93e69a.tar.xz |
silence gcc array-bounds warning
In shorten_unambiguous_ref, we build and cache a reverse-map of the
rev-parse rules like this:
static char **scanf_fmts;
static int nr_rules;
if (!nr_rules) {
for (; ref_rev_parse_rules[nr_rules]; nr_rules++)
... generate scanf_fmts ...
}
where ref_rev_parse_rules is terminated with a NULL pointer.
Compiling with "gcc -O2 -Wall" does not cause any problems, but
compiling with "-O3 -Wall" generates:
$ make CFLAGS='-O3 -Wall' refs.o
refs.c: In function ‘shorten_unambiguous_ref’:
refs.c:3379:29: warning: array subscript is above array bounds [-Warray-bounds]
for (; ref_rev_parse_rules[nr_rules]; nr_rules++)
Curiously, we can silence this by explicitly nr_rules to 0
in the beginning of the loop, even though the compiler
should be able to tell that we follow this code path only
when nr_rules is already 0.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/RelNotes/1.6.6.txt')
0 files changed, 0 insertions, 0 deletions