From face8e0a7d5f530ee3e5e63ab1e3d6ecd497326b Mon Sep 17 00:00:00 2001 From: Joel Klinghed Date: Sat, 22 Jul 2017 01:26:18 +0200 Subject: Fix warnings for GCC as well GCC ignores __attribute__((unused)) when compiling C++ code. (Yes, this is true) GCC -Wmaybe-uninitialized just gives false positives in my experience, so drop it. However, this exposed a problem with AX_APPEND_COMPILE_FLAGS and clang. clang gives a warning, but not an error, when given a -Wno-UNKNOWN-FLAG. So add -Werror as an extra flag to make clang fail on unknown -Wno-UNKNOWN-FLAG --- configure.ac | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'configure.ac') diff --git a/configure.ac b/configure.ac index 53d0306..7564309 100644 --- a/configure.ac +++ b/configure.ac @@ -46,7 +46,9 @@ CXXFLAGS="$OLDCXXFLAGS" AX_CXXFLAGS_WARN_ALL([DEFINES]) AX_APPEND_COMPILE_FLAGS([-fno-rtti -fno-exceptions],DEFINES) -AX_APPEND_COMPILE_FLAGS([-Wextra -Wno-missing-field-initializers],DEFINES) +extra_flags= +AX_CHECK_COMPILE_FLAG([-Werror], [extra_flags=-Werror]) +AX_APPEND_COMPILE_FLAGS([-Wextra -Wno-missing-field-initializers -Wno-maybe-uninitialized],DEFINES,[$extra_flags]) AC_ARG_ENABLE([debug], AC_HELP_STRING([compile with debug options]), if test "x$enableval" = "xyes"; then DEFINES="$DEFINES -g -O0 -DDEBUG" -- cgit v1.2.3-70-g09d2