Signed-off-by: Johan Herland <johan@xxxxxxxxxxx> --- On Monday 21 May 2007, Johannes Schindelin wrote: > > We do not appreciate C99 initializers, declarations after statements, > or "0" instead of "NULL". > > [...] > > +Also, variables have to be declared at the beginning of the block > +(you can check this with gcc, using the -Wdeclaration-after-statement > +option). Why not automatically enforce it by putting -Wdeclaration-after-statement in the Makefile? It should probably be protected by some GCC if-ery, but then again, so should this: CC = gcc Have fun! ...Johan Makefile | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 29243c6..4e91516 100644 --- a/Makefile +++ b/Makefile @@ -135,7 +135,7 @@ uname_P := $(shell sh -c 'uname -p 2>/dev/null || echo not') # CFLAGS and LDFLAGS are for the users to override from the command line. -CFLAGS = -g -O2 -Wall +CFLAGS = -g -O2 -Wall -Wdeclaration-after-statement LDFLAGS = ALL_CFLAGS = $(CFLAGS) ALL_LDFLAGS = $(LDFLAGS) -- 1.5.2.101.gee49f - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
