Earlier we added "-Wformat-security" to developer builds, assuming that "-Wall" (which includes "-Wformat" which in turn is required to use "-Wformat-security") is always in effect. This is not true when config.mak.autogen is in use, unfortunately. This has been fixed by unconditionally adding "-Wall" to developer builds. * jk/dev-build-format-security: config.mak.dev: add -Wall, primarily for -Wformat, to help autoconf users
		
			
				
	
	
		
			49 lines
		
	
	
		
			1.4 KiB
		
	
	
	
		
			Plaintext
		
	
	
	
	
	
			
		
		
	
	
			49 lines
		
	
	
		
			1.4 KiB
		
	
	
	
		
			Plaintext
		
	
	
	
	
	
ifeq ($(filter no-error,$(DEVOPTS)),)
 | 
						|
CFLAGS += -Werror
 | 
						|
endif
 | 
						|
ifneq ($(filter pedantic,$(DEVOPTS)),)
 | 
						|
CFLAGS += -pedantic
 | 
						|
# don't warn for each N_ use
 | 
						|
CFLAGS += -DUSE_PARENS_AROUND_GETTEXT_N=0
 | 
						|
endif
 | 
						|
CFLAGS += -Wall
 | 
						|
CFLAGS += -Wdeclaration-after-statement
 | 
						|
CFLAGS += -Wformat-security
 | 
						|
CFLAGS += -Wno-format-zero-length
 | 
						|
CFLAGS += -Wold-style-definition
 | 
						|
CFLAGS += -Woverflow
 | 
						|
CFLAGS += -Wpointer-arith
 | 
						|
CFLAGS += -Wstrict-prototypes
 | 
						|
CFLAGS += -Wunused
 | 
						|
CFLAGS += -Wvla
 | 
						|
 | 
						|
ifndef COMPILER_FEATURES
 | 
						|
COMPILER_FEATURES := $(shell ./detect-compiler $(CC))
 | 
						|
endif
 | 
						|
 | 
						|
ifneq ($(filter clang4,$(COMPILER_FEATURES)),)
 | 
						|
CFLAGS += -Wtautological-constant-out-of-range-compare
 | 
						|
endif
 | 
						|
 | 
						|
ifneq ($(or $(filter gcc6,$(COMPILER_FEATURES)),$(filter clang4,$(COMPILER_FEATURES))),)
 | 
						|
CFLAGS += -Wextra
 | 
						|
# if a function is public, there should be a prototype and the right
 | 
						|
# header file should be included. If not, it should be static.
 | 
						|
CFLAGS += -Wmissing-prototypes
 | 
						|
ifeq ($(filter extra-all,$(DEVOPTS)),)
 | 
						|
# These are disabled because we have these all over the place.
 | 
						|
CFLAGS += -Wno-empty-body
 | 
						|
CFLAGS += -Wno-missing-field-initializers
 | 
						|
CFLAGS += -Wno-sign-compare
 | 
						|
CFLAGS += -Wno-unused-parameter
 | 
						|
endif
 | 
						|
endif
 | 
						|
 | 
						|
# uninitialized warnings on gcc 4.9.2 in xdiff/xdiffi.c and config.c
 | 
						|
# not worth fixing since newer compilers correctly stop complaining
 | 
						|
ifneq ($(filter gcc4,$(COMPILER_FEATURES)),)
 | 
						|
ifeq ($(filter gcc5,$(COMPILER_FEATURES)),)
 | 
						|
CFLAGS += -Wno-uninitialized
 | 
						|
endif
 | 
						|
endif
 |