diff options
author | Guy Harris <guy@alum.mit.edu> | 2014-04-20 15:44:53 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2014-04-20 22:47:18 +0000 |
commit | 68f9811b1bed1397d2ba06af63488fe16bc5bb5e (patch) | |
tree | bf1c124437efd4d3c1c37a4da10b926aac9b3cc5 /acinclude.m4 | |
parent | 62592ee5ec81b78c5cd9755d7d984e7c72722c9a (diff) |
Don't distinguish between "GCC" and "Clang" extra -W flags.
We test whether a given compiler supports a given -W flag, so we don't
need to separate them and check them only for particular compilers.
To make that even clearer, rename the --enable option from
--enable-extra-gcc-checks to --enable-extra-compiler-checks, and
document it as just "do additional -W checks", and rename the
WIRESHARK_EXTRA_GCC_ CMake variables to WIRESHARK_EXTRA_COMPILER_.
Sync up the lists of warning flags in CMake with the lists in autoconf.
Uncomment -Wdocumentation while we're at it. If it doesn't work *at
all*, comment it out until it's fixed, or, better yet, fix it; if it
still produces warnings, we just leave it among the "extra" flags.
Change-Id: I4042affdade612e4025e2881d08f1ca69d759626
Reviewed-on: https://code.wireshark.org/review/1226
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'acinclude.m4')
-rw-r--r-- | acinclude.m4 | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/acinclude.m4 b/acinclude.m4 index 3c9ec8be1c..136fc27914 100644 --- a/acinclude.m4 +++ b/acinclude.m4 @@ -2219,7 +2219,7 @@ CC is not __clang__ ]])], [ CC_IS_CLANG='yes' - CFLAGS="$CFLAGS -Qunused-arguments" + CFLAGS="$CFLAGS" ], CC_IS_CLANG='no' ) |