diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2018-07-01 07:08:32 -0700 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2018-07-01 07:09:08 -0700 |
commit | f24fbe9d9297f57b632fbefd87770b4f39a611ef (patch) | |
tree | 6beccc7e2bd428be883b19d476e758698c576776 | |
parent | e09e64c7e0d3c9d0cca82cb80713f8aea195d493 (diff) | |
download | gnulib-f24fbe9d9297f57b632fbefd87770b4f39a611ef.tar.gz |
manywarnings: omit -Wswitch-default
This should make things more consistent, as we already ignore
-Wswitch-enum. Problem reported by Reuben Thomas; see:
https://lists.gnu.org/r/bug-gnulib/2018-05/msg00179.html
* build-aux/g++-warning.spec, build-aux/gcc-warning.spec:
Add -Wswitch-default.
* m4/manywarnings-c++.m4 (gl_MANYWARN_ALL_GCC_CXX_IMPL):
* m4/manywarnings.m4 (gl_MANYWARN_ALL_GCC):
Remove -Wswitch-default.
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | build-aux/g++-warning.spec | 1 | ||||
-rw-r--r-- | build-aux/gcc-warning.spec | 1 | ||||
-rw-r--r-- | m4/manywarnings-c++.m4 | 3 | ||||
-rw-r--r-- | m4/manywarnings.m4 | 3 |
5 files changed, 14 insertions, 4 deletions
@@ -1,5 +1,15 @@ 2018-07-01 Paul Eggert <eggert@cs.ucla.edu> + manywarnings: omit -Wswitch-default + This should make things more consistent, as we already ignore + -Wswitch-enum. Problem reported by Reuben Thomas; see: + https://lists.gnu.org/r/bug-gnulib/2018-05/msg00179.html + * build-aux/g++-warning.spec, build-aux/gcc-warning.spec: + Add -Wswitch-default. + * m4/manywarnings-c++.m4 (gl_MANYWARN_ALL_GCC_CXX_IMPL): + * m4/manywarnings.m4 (gl_MANYWARN_ALL_GCC): + Remove -Wswitch-default. + regex: revert most trimming Problems reported by Bruno Haible in: https://lists.gnu.org/r/bug-gnulib/2018-07/msg00001.html diff --git a/build-aux/g++-warning.spec b/build-aux/g++-warning.spec index ae3e86676d..99ef3c8a8d 100644 --- a/build-aux/g++-warning.spec +++ b/build-aux/g++-warning.spec @@ -82,6 +82,7 @@ -Wstrict-prototypes c -Wstrict-selector-match objc and objc++ -Wsurprising fortran +-Wswitch-default https://lists.gnu.org/r/bug-gnulib/2018-05/msg00179.html -Wswitch-enum FIXME maybe? borderline. some will want this -Wsynth deprecated -Wtabs fortran diff --git a/build-aux/gcc-warning.spec b/build-aux/gcc-warning.spec index c47298e93c..8c0a3dc750 100644 --- a/build-aux/gcc-warning.spec +++ b/build-aux/gcc-warning.spec @@ -121,6 +121,7 @@ -Wsubobject-linkage c++ and objc++ -Wsuggest-override c++ and objc++ -Wsurprising fortran +-Wswitch-default https://lists.gnu.org/r/bug-gnulib/2018-05/msg00179.html -Wswitch-enum FIXME maybe? borderline. some will want this -Wsynth deprecated -Wtabs fortran diff --git a/m4/manywarnings-c++.m4 b/m4/manywarnings-c++.m4 index 3a4797d82a..28d9aa0d48 100644 --- a/m4/manywarnings-c++.m4 +++ b/m4/manywarnings-c++.m4 @@ -1,4 +1,4 @@ -# manywarnings-c++.m4 serial 1 +# manywarnings-c++.m4 serial 2 dnl Copyright (C) 2008-2018 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -180,7 +180,6 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC_CXX_IMPL], -Wsuggest-override \ -Wswitch \ -Wswitch-bool \ - -Wswitch-default \ -Wsync-nand \ -Wsystem-headers \ -Wtrampolines \ diff --git a/m4/manywarnings.m4 b/m4/manywarnings.m4 index 925c40e139..516c587476 100644 --- a/m4/manywarnings.m4 +++ b/m4/manywarnings.m4 @@ -1,4 +1,4 @@ -# manywarnings.m4 serial 15 +# manywarnings.m4 serial 16 dnl Copyright (C) 2008-2018 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -239,7 +239,6 @@ m4_defun([gl_MANYWARN_ALL_GCC(C)], -Wsuggest-final-types \ -Wswitch \ -Wswitch-bool \ - -Wswitch-default \ -Wswitch-unreachable \ -Wsync-nand \ -Wsystem-headers \ |