Reenable -Wstrict-prototypes

One part of this patch reverts commit 22bc9a46, which disabled the
warning. The rest of it deals with the warning by adding a #pragma for
newer gcc and by disabling -Werror for compilers that can't deal with
the #pragma.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-id: 1361563731-13307-1-git-send-email-kwolf@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
Kevin Wolf 2013-02-22 21:08:51 +01:00 committed by Anthony Liguori
parent 5cbb082837
commit c95e3080a4
2 changed files with 24 additions and 7 deletions

19
configure vendored
View File

@ -284,7 +284,7 @@ sdl_config="${SDL_CONFIG-${cross_prefix}sdl-config}"
# default flags for all hosts # default flags for all hosts
QEMU_CFLAGS="-fno-strict-aliasing $QEMU_CFLAGS" QEMU_CFLAGS="-fno-strict-aliasing $QEMU_CFLAGS"
QEMU_CFLAGS="-Wall -Wundef -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS" QEMU_CFLAGS="-Wall -Wundef -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
QEMU_CFLAGS="-Wredundant-decls $QEMU_CFLAGS" QEMU_CFLAGS="-Wstrict-prototypes -Wredundant-decls $QEMU_CFLAGS"
QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS" QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS"
QEMU_INCLUDES="-I. -I\$(SRC_PATH) -I\$(SRC_PATH)/include" QEMU_INCLUDES="-I. -I\$(SRC_PATH) -I\$(SRC_PATH)/include"
if test "$debug_info" = "yes"; then if test "$debug_info" = "yes"; then
@ -3115,20 +3115,27 @@ if compile_prog "" "" ; then
fi fi
######################################## ########################################
# check whether we can disable the -Wunused-but-set-variable # check whether we can disable warning option with a pragma (this is needed
# option with a pragma (this is needed to silence a warning in # to silence warnings in the headers of some versions of external libraries).
# some versions of the valgrind VALGRIND_STACK_DEREGISTER macro.) # This test has to be compiled with -Werror as otherwise an unknown pragma is
# This test has to be compiled with -Werror as otherwise an # only a warning.
# unknown pragma is only a warning. #
# If we can't selectively disable warning in the code, disable -Werror so that
# the build doesn't fail anyway.
pragma_disable_unused_but_set=no pragma_disable_unused_but_set=no
cat > $TMPC << EOF cat > $TMPC << EOF
#pragma GCC diagnostic ignored "-Wunused-but-set-variable" #pragma GCC diagnostic ignored "-Wunused-but-set-variable"
#pragma GCC diagnostic ignored "-Wstrict-prototypes"
int main(void) { int main(void) {
return 0; return 0;
} }
EOF EOF
if compile_prog "-Werror" "" ; then if compile_prog "-Werror" "" ; then
pragma_diagnostic_available=yes pragma_diagnostic_available=yes
else
werror=no
fi fi
######################################## ########################################

View File

@ -34,7 +34,18 @@
#define GETTEXT_PACKAGE "qemu" #define GETTEXT_PACKAGE "qemu"
#define LOCALEDIR "po" #define LOCALEDIR "po"
#include "qemu-common.h"
#ifdef CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE
/* Work around an -Wstrict-prototypes warning in GTK headers */
#pragma GCC diagnostic ignored "-Wstrict-prototypes"
#endif
#include <gtk/gtk.h> #include <gtk/gtk.h>
#ifdef CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE
#pragma GCC diagnostic error "-Wstrict-prototypes"
#endif
#include <gdk/gdkkeysyms.h> #include <gdk/gdkkeysyms.h>
#include <glib/gi18n.h> #include <glib/gi18n.h>
#include <locale.h> #include <locale.h>
@ -46,7 +57,6 @@
#include <pty.h> #include <pty.h>
#include <math.h> #include <math.h>
#include "qemu-common.h"
#include "ui/console.h" #include "ui/console.h"
#include "sysemu/sysemu.h" #include "sysemu/sysemu.h"
#include "qmp-commands.h" #include "qmp-commands.h"