From 8bd1b254a3b5e25f2da0f7f245ba1314f9e3e163 Mon Sep 17 00:00:00 2001 From: Martin Quinson Date: Tue, 21 Jun 2016 21:58:02 +0200 Subject: [PATCH] in NDEBUG, show the warnings but don't die --- tools/cmake/GCCFlags.cmake | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/tools/cmake/GCCFlags.cmake b/tools/cmake/GCCFlags.cmake index f71ff39fe0..0a1719720d 100644 --- a/tools/cmake/GCCFlags.cmake +++ b/tools/cmake/GCCFlags.cmake @@ -13,8 +13,8 @@ set(warnCFLAGS "") set(optCFLAGS "") set(warnCXXFLAGS "") -if(enable_compile_warnings AND enable_debug) - set(warnCFLAGS "-fno-common -Wall -Wunused -Wmissing-declarations -Wpointer-arith -Wchar-subscripts -Wcomment -Wformat -Wwrite-strings -Wno-unused-function -Wno-unused-parameter -Wno-strict-aliasing -Wno-format-nonliteral -Werror") +if(enable_compile_warnings) + set(warnCFLAGS "-fno-common -Wall -Wunused -Wmissing-declarations -Wpointer-arith -Wchar-subscripts -Wcomment -Wformat -Wwrite-strings -Wno-unused-function -Wno-unused-parameter -Wno-strict-aliasing -Wno-format-nonliteral") if(CMAKE_COMPILER_IS_GNUCC) set(warnCFLAGS "${warnCFLAGS} -Wclobbered -Wno-error=clobbered -Wno-unused-local-typedefs -Wno-error=attributes") endif() @@ -34,6 +34,11 @@ if(enable_compile_warnings AND enable_debug) set(CMAKE_JAVA_COMPILE_FLAGS "-Xlint") endif() +# NDEBUG gives a lot of "initialized but unused variables" errors. Don't die anyway. +if(enable_compile_warnings AND enable_debug) + set(warnCFLAGS "${warnCFLAGS} -Werror") +endif() + # Activate the warnings on #if FOOBAR when FOOBAR has no value # It breaks on FreeBSD within Boost headers, so activate this only in Pure Hardcore debug mode. if(enable_maintainer_mode) -- 2.30.2