From: Pedro Alves Date: Sat, 28 Feb 2015 19:11:10 +0000 (+0000) Subject: Remove C-specific warnings from common warning set X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=aac331e484bba9736f12602da3715c8f7e0f1a45;p=deliverable%2Fbinutils-gdb.git Remove C-specific warnings from common warning set Whoops, these are C specific, but I somehow missed the warnings before: cc1plus: warning: command line option ‘-Wmissing-prototypes’ is valid for C/ObjC but not for C++ [enabled by default] cc1plus: warning: command line option ‘-Wdeclaration-after-statement’ is valid for C/ObjC but not for C++ [enabled by default] cc1plus: warning: command line option ‘-Wmissing-parameter-type’ is valid for C/ObjC but not for C++ [enabled by default] cc1plus: warning: command line option ‘-Wold-style-declaration’ is valid for C/ObjC but not for C++ [enabled by default] cc1plus: warning: command line option ‘-Wold-style-definition’ is valid for C/ObjC but not for C++ [enabled by default] gdb/ChangeLog: 2015-03-07 Pedro Alves * configure.ac (build_warnings): Move -Wmissing-prototypes -Wdeclaration-after-statement -Wmissing-parameter-type -Wold-style-declaration -Wold-style-definition to the C-specific set. * configure: Regenerate. gdb/gdbserver/ChangeLog: 2015-03-07 Pedro Alves * configure.ac (build_warnings): Move -Wdeclaration-after-statement to the C-specific set. * configure: Regenerate. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 522f2bb4cd..eee5badd29 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2015-03-07 Pedro Alves + + * configure.ac (build_warnings): Move -Wmissing-prototypes + -Wdeclaration-after-statement -Wmissing-parameter-type + -Wold-style-declaration -Wold-style-definition to the C-specific + set. + * configure: Regenerate. + 2015-03-07 Pedro Alves * common/gdb_socket.h: New file. diff --git a/gdb/configure b/gdb/configure index 3297d37c5f..1ddcc62ab8 100755 --- a/gdb/configure +++ b/gdb/configure @@ -13383,18 +13383,19 @@ if test "${ERROR_ON_WARNING}" = yes ; then fi # These options work in either C or C++ modes. -build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ +build_warnings="-Wall -Wpointer-arith \ -Wno-unused -Wunused-value -Wunused-function \ --Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ --Wdeclaration-after-statement -Wempty-body -Wmissing-parameter-type \ --Wold-style-declaration -Wold-style-definition" +-Wno-switch -Wno-char-subscripts \ +-Wempty-body" # Now add in C and C++ specific options, depending on mode. if test "$enable_build_with_cxx" = "yes"; then build_warnings="$build_warnings -Wno-sign-compare -Wno-write-strings \ -Wno-narrowing" else - build_warnings="$build_warnings -Wpointer-sign" + build_warnings="$build_warnings -Wpointer-sign -Wmissing-prototypes \ +-Wdeclaration-after-statement -Wmissing-parameter-type \ +-Wold-style-declaration -Wold-style-definition" fi # Enable -Wno-format by default when using gcc on mingw since many diff --git a/gdb/configure.ac b/gdb/configure.ac index 0898d0e145..a4886c93ed 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -1978,18 +1978,19 @@ if test "${ERROR_ON_WARNING}" = yes ; then fi # These options work in either C or C++ modes. -build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ +build_warnings="-Wall -Wpointer-arith \ -Wno-unused -Wunused-value -Wunused-function \ --Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ --Wdeclaration-after-statement -Wempty-body -Wmissing-parameter-type \ --Wold-style-declaration -Wold-style-definition" +-Wno-switch -Wno-char-subscripts \ +-Wempty-body" # Now add in C and C++ specific options, depending on mode. if test "$enable_build_with_cxx" = "yes"; then build_warnings="$build_warnings -Wno-sign-compare -Wno-write-strings \ -Wno-narrowing" else - build_warnings="$build_warnings -Wpointer-sign" + build_warnings="$build_warnings -Wpointer-sign -Wmissing-prototypes \ +-Wdeclaration-after-statement -Wmissing-parameter-type \ +-Wold-style-declaration -Wold-style-definition" fi # Enable -Wno-format by default when using gcc on mingw since many diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 8aa38a301b..b27090ee9a 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,9 @@ +2015-03-07 Pedro Alves + + * configure.ac (build_warnings): Move + -Wdeclaration-after-statement to the C-specific set. + * configure: Regenerate. + 2015-03-07 Pedro Alves * gdbreplay.c: No longer include , , diff --git a/gdb/gdbserver/configure b/gdb/gdbserver/configure index a14def95ac..cbf4b4ebf4 100755 --- a/gdb/gdbserver/configure +++ b/gdb/gdbserver/configure @@ -5794,13 +5794,15 @@ if test "${ERROR_ON_WARNING}" = yes ; then WERROR_CFLAGS="-Werror" fi -build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ +build_warnings="-Wall -Wpointer-arith \ -Wformat-nonliteral -Wno-char-subscripts -Wempty-body" # Now add in C and C++ specific options, depending on mode. if test "$enable_build_with_cxx" = "yes"; then build_warnings="$build_warnings -Wno-sign-compare -Wno-write-strings \ -Wno-narrowing" +else + build_warnings="$build_warnings -Wdeclaration-after-statement" fi # The set of warnings supported by a C++ compiler is not the same as diff --git a/gdb/gdbserver/configure.ac b/gdb/gdbserver/configure.ac index cefc69e3ce..0381aa986d 100644 --- a/gdb/gdbserver/configure.ac +++ b/gdb/gdbserver/configure.ac @@ -167,13 +167,15 @@ if test "${ERROR_ON_WARNING}" = yes ; then WERROR_CFLAGS="-Werror" fi -build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ +build_warnings="-Wall -Wpointer-arith \ -Wformat-nonliteral -Wno-char-subscripts -Wempty-body" # Now add in C and C++ specific options, depending on mode. if test "$enable_build_with_cxx" = "yes"; then build_warnings="$build_warnings -Wno-sign-compare -Wno-write-strings \ -Wno-narrowing" +else + build_warnings="$build_warnings -Wdeclaration-after-statement" fi # The set of warnings supported by a C++ compiler is not the same as