| ifeq ($(filter no-error,$(DEVOPTS)),) | |
| DEVELOPER_CFLAGS += -Werror | |
| endif | |
| ifneq ($(filter pedantic,$(DEVOPTS)),) | |
| DEVELOPER_CFLAGS += -pedantic | |
| # don't warn for each N_ use | |
| DEVELOPER_CFLAGS += -DUSE_PARENS_AROUND_GETTEXT_N=0 | |
| endif | |
| DEVELOPER_CFLAGS += -Wall | |
| DEVELOPER_CFLAGS += -Wdeclaration-after-statement | |
| DEVELOPER_CFLAGS += -Wformat-security | |
| DEVELOPER_CFLAGS += -Wold-style-definition | |
| DEVELOPER_CFLAGS += -Woverflow | |
| DEVELOPER_CFLAGS += -Wpointer-arith | |
| DEVELOPER_CFLAGS += -Wstrict-prototypes | |
| DEVELOPER_CFLAGS += -Wunused | |
| DEVELOPER_CFLAGS += -Wvla | |
| DEVELOPER_CFLAGS += -DENABLE_SHA256 | |
| ifndef COMPILER_FEATURES | |
| COMPILER_FEATURES := $(shell ./detect-compiler $(CC)) | |
| endif | |
| ifneq ($(filter clang4,$(COMPILER_FEATURES)),) | |
| DEVELOPER_CFLAGS += -Wtautological-constant-out-of-range-compare | |
| endif | |
| ifneq ($(or $(filter gcc6,$(COMPILER_FEATURES)),$(filter clang4,$(COMPILER_FEATURES))),) | |
| DEVELOPER_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. | |
| DEVELOPER_CFLAGS += -Wmissing-prototypes | |
| ifeq ($(filter extra-all,$(DEVOPTS)),) | |
| # These are disabled because we have these all over the place. | |
| DEVELOPER_CFLAGS += -Wno-empty-body | |
| DEVELOPER_CFLAGS += -Wno-missing-field-initializers | |
| DEVELOPER_CFLAGS += -Wno-sign-compare | |
| DEVELOPER_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)),) | |
| DEVELOPER_CFLAGS += -Wno-uninitialized | |
| endif | |
| endif |