>From d4cbfaeca1e838a0d0373adfbd133a9f5eaa8e87 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Tue, 22 Oct 2019 11:34:56 -0700 Subject: [PATCH 1/5] build: re-enable type-limits checking * configure.ac: When --enable-gcc-warnings is used, omit -Wno-type-limits. The need for -Wno-type-limits has passed, now that intprops.h uses builtin primitives for GCC 5 and later, given that recent GCCs issue type-limits warnings only for non-constant expressions. --enable-gcc-warnings is not intended for use with old compilers, so we can drop -Wno-type-limits now. --- configure.ac | 2 -- 1 file changed, 2 deletions(-) diff --git a/configure.ac b/configure.ac index d90c710e3..292ae0bf2 100644 --- a/configure.ac +++ b/configure.ac @@ -134,7 +134,6 @@ if test "$gl_gcc_warnings" = yes; then nw="$nw -Wswitch-enum" # Too many warnings for now nw="$nw -Wswitch-default" # Too many warnings for now nw="$nw -Wstack-protector" # not worth working around - nw="$nw -Wtype-limits" # False alarms for portable code nw="$nw -Wformat-overflow=2" # False alarms due to GCC bug 80776 nw="$nw -Wformat-truncation=2" # False alarm in ls.c, probably related # things I might fix soon: @@ -155,7 +154,6 @@ if test "$gl_gcc_warnings" = yes; then gl_WARN_ADD([$w]) done gl_WARN_ADD([-Wno-sign-compare]) # Too many warnings for now - gl_WARN_ADD([-Wno-type-limits]) # False alarms for portable code gl_WARN_ADD([-Wno-unused-parameter]) # Too many warnings for now gl_WARN_ADD([-Wno-format-nonliteral]) -- 2.21.0