From 6c7443a65369d0bd236d9eb19e59df6639c91eec Mon Sep 17 00:00:00 2001 From: David du Colombier <0intro@gmail.com> Date: Tue, 21 Oct 2014 14:22:12 +0200 Subject: fix clang 3.4 warnings and ignore uninteresting ones fixed warnings: src/cmd/fossil/disk.c:37:14: warning: use of GNU 'missing =' extension in designator [-Wgnu-designator] src/cmd/fossil/disk.c:38:14: warning: use of GNU 'missing =' extension in designator [-Wgnu-designator] src/cmd/fossil/disk.c:39:14: warning: use of GNU 'missing =' extension in designator [-Wgnu-designator] src/cmd/fossil/disk.c:40:13: warning: use of GNU 'missing =' extension in designator [-Wgnu-designator] src/cmd/fossil/disk.c:41:14: warning: use of GNU 'missing =' extension in designator [-Wgnu-designator] src/libndb/ndbreorder.c:41:55: warning: for loop has empty body [-Wempty-body] ignored warnings: src/cmd/acid/dbg.y:393:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/bc.y:1327:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/bc.y:1327:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/grep/grep.y:420:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/grep/grep.y:420:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/hoc/hoc.y:692:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/hoc/hoc.y:692:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/lex/parser.y:886:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/rc/syn.y:303:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/cmd/units.y:1003:9: warning: array index -1 is before the beginning of the array [-Warray-bounds] src/libregexp/regcomp.c:19:16: warning: variable 'reprog' is not needed and will not be emitted [-Wunneeded-internal-declaration] LGTM=rsc R=rsc https://codereview.appspot.com/158250043 --- bin/9c | 2 ++ src/cmd/fossil/disk.c | 10 +++++----- src/libndb/ndbreorder.c | 2 +- 3 files changed, 8 insertions(+), 6 deletions(-) diff --git a/bin/9c b/bin/9c index 85aa0822..64d5c3a2 100755 --- a/bin/9c +++ b/bin/9c @@ -67,6 +67,8 @@ useclang() -Wno-unused-value \ -Wno-array-bounds \ -Wno-gnu-designator \ + -Wno-array-bounds \ + -Wno-unneeded-internal-declaration \ -fsigned-char \ -fno-caret-diagnostics \ " diff --git a/src/cmd/fossil/disk.c b/src/cmd/fossil/disk.c index 04ea5cdd..61370dd8 100644 --- a/src/cmd/fossil/disk.c +++ b/src/cmd/fossil/disk.c @@ -34,11 +34,11 @@ struct Disk { /* keep in sync with Part* enum in dat.h */ static char *partname[] = { - [PartError] "error", - [PartSuper] "super", - [PartLabel] "label", - [PartData] "data", - [PartVenti] "venti", + [PartError] = "error", + [PartSuper] = "super", + [PartLabel] = "label", + [PartData] = "data", + [PartVenti] = "venti", }; Disk * diff --git a/src/libndb/ndbreorder.c b/src/libndb/ndbreorder.c index 167d0a0a..390d7818 100644 --- a/src/libndb/ndbreorder.c +++ b/src/libndb/ndbreorder.c @@ -38,7 +38,7 @@ ndbreorder(Ndbtuple *t, Ndbtuple *x) if(x != last->line){ /* find entry before x */ - for(prev = last; prev->line != x; prev = prev->line); + for(prev = last; prev->line != x; prev = prev->line) ; /* detach line */ -- cgit v1.2.3