diff options
authorGravatar Denis Vlasenko <vda.linux@googlemail.com>2008-05-15 21:44:46 +0000
committerGravatar Denis Vlasenko <vda.linux@googlemail.com>2008-05-15 21:44:46 +0000
commite251337a4d605e22715ed28b0a168b7fe18c4c9f (patch)
parent6b06cb80be64fcf207bee734cc678c25434ed1a4 (diff)
Add more -W options to gcc.
1 files changed, 3 insertions, 14 deletions
diff --git a/Makefile.flags b/Makefile.flags
index f27c601c8..3889df09b 100644
--- a/Makefile.flags
+++ b/Makefile.flags
@@ -21,27 +21,16 @@ CPPFLAGS += \
CFLAGS += $(call cc-option,-Wall -Wshadow -Wwrite-strings,)
CFLAGS += $(call cc-option,-Wundef -Wstrict-prototypes,)
CFLAGS += $(call cc-option,-Wunused -Wunused-parameter,)
-# If you want to add "-Wmissing-prototypes -Wmissing-declarations" above
-# (or anything else for that matter) make sure that it is still possible
-# to build bbox without warnings. Current offender: find.c:alloc_action().
-# Looks more like gcc bug: gcc will warn on it with or without prototype.
-# But still, warning-free compile is a must, or else we will drown
-# in warnings pretty soon.
+CFLAGS += $(call cc-option,-Wmissing-prototypes -Wmissing-declarations,)
+# If you want to add more -Wsomething above, make sure that it is
+# still possible to build bbox without warnings.
ifeq ($(CONFIG_WERROR),y)
CFLAGS += $(call cc-option,-Werror,)
-# for development, warn a little bit about unused results..
# gcc 3.x emits bogus "old style proto" warning on find.c:alloc_action()
CFLAGS += $(call cc-ifversion, -ge, 0400, -Wold-style-definition)
-# gcc emits bogus "no prev proto" warning on find.c:alloc_action()
-ifneq ($(CONFIG_WERROR),y)
-CFLAGS += $(call cc-option,-Wmissing-prototypes -Wmissing-declarations,)
CFLAGS += $(call cc-option,-Os -fno-builtin-strlen -finline-limit=0 -fomit-frame-pointer -ffunction-sections -fdata-sections,)
# -fno-guess-branch-probability: prohibit pseudo-random guessing
# of branch probabilities (hopefully makes bloatcheck more stable):