summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFilipe Brandenburger <filbranden@google.com>2015-06-10 22:40:31 -0700
committerFilipe Brandenburger <filbranden@google.com>2015-06-10 22:45:06 -0700
commitf73fb7b742f294b6d2126afa16001bd2ff6ab461 (patch)
treece38f2534a45ab9228c7c4d7c5507cc51bd9b774
parent37c47e5e5b8e47dd886ed6e8355b1203082b4a3e (diff)
build-sys: suppress warnings of unused results
Unfortunately, gcc keeps warning about those even when we use an explicit (void) cast to indicate we are not interested in the result. LLVM's clang does not have that issue and works fine with the casts. GCC bug being tracked at: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425 Until that GCC bug is fixed (and the version with the fix is in many/most distributions) or we switch to LLVM as the default compiler, it looks like we'll have to disable this warning by default... Tested by building files known to present warnings about unused results without the suppression, confirmed that the warnings were no longer present with this patch applied. This partially reverts commit 00c11bc53a1d37 ("build-sys: don't suppress irrelevant warnings").
-rw-r--r--configure.ac1
1 files changed, 1 insertions, 0 deletions
diff --git a/configure.ac b/configure.ac
index 878b23b061..344bfe1e21 100644
--- a/configure.ac
+++ b/configure.ac
@@ -173,6 +173,7 @@ CC_CHECK_FLAGS_APPEND([with_cflags], [CFLAGS], [\
-Wwrite-strings \
-Wno-unused-parameter \
-Wno-missing-field-initializers \
+ -Wno-unused-result \
-Wno-format-signedness \
-Werror=overflow \
-Wdate-time \