diff options
| author | Filipe Brandenburger <filbranden@google.com> | 2015-06-10 22:40:31 -0700 |
|---|---|---|
| committer | Filipe Brandenburger <filbranden@google.com> | 2015-06-10 22:45:06 -0700 |
| commit | f73fb7b742f294b6d2126afa16001bd2ff6ab461 (patch) | |
| tree | ce38f2534a45ab9228c7c4d7c5507cc51bd9b774 | |
| parent | 37c47e5e5b8e47dd886ed6e8355b1203082b4a3e (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.ac | 1 |
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 \ |