[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 15/15] configure: Enable extra compiler warnings
From: |
Michael Tokarev |
Subject: |
[Qemu-devel] [PULL 15/15] configure: Enable extra compiler warnings |
Date: |
Sat, 14 Sep 2013 23:47:41 +0400 |
From: Stefan Weil <address@hidden>
Compiler option -Wextra enables an additional set of compiler warnings.
Some of these warnings were already enabled explicitly in QEMU:
-Wold-style-declaration, -Wtype-limits, -Wignored-qualifiers and
-Wempty-body are now redundant and can be removed.
Others don't work with the current code and must be disabled to
avoid warnings: -Wno-missing-field-initializers, -Wno-override-init,
-Wno-sign-compare and -Wno-unused-parameter.
Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>
---
configure | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/configure b/configure
index 21451ba..a17cac7 100755
--- a/configure
+++ b/configure
@@ -310,7 +310,7 @@ sdl_config="${SDL_CONFIG-${cross_prefix}sdl-config}"
# default flags for all hosts
QEMU_CFLAGS="-fno-strict-aliasing $QEMU_CFLAGS"
-QEMU_CFLAGS="-Wall -Wundef -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
+QEMU_CFLAGS="-Wall -Wextra -Wundef -Wwrite-strings -Wmissing-prototypes
$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_INCLUDES="-I. -I\$(SRC_PATH) -I\$(SRC_PATH)/include"
@@ -1236,12 +1236,16 @@ if test -z "$werror" ; then
fi
fi
-gcc_flags="-Wold-style-declaration -Wold-style-definition -Wtype-limits"
-gcc_flags="-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers
$gcc_flags"
-gcc_flags="-Wmissing-include-dirs -Wempty-body -Wnested-externs $gcc_flags"
+gcc_flags="-Wold-style-definition"
+gcc_flags="-Wformat-security -Wformat-y2k -Winit-self $gcc_flags"
+gcc_flags="-Wmissing-include-dirs -Wnested-externs $gcc_flags"
gcc_flags="-Wendif-labels $gcc_flags"
gcc_flags="-Wno-initializer-overrides $gcc_flags"
+gcc_flags="-Wno-missing-field-initializers $gcc_flags"
+gcc_flags="-Wno-override-init $gcc_flags"
+gcc_flags="-Wno-sign-compare $gcc_flags"
gcc_flags="-Wno-string-plus-int $gcc_flags"
+gcc_flags="-Wno-unused-parameter $gcc_flags"
# Note that we do not add -Werror to gcc_flags here, because that would
# enable it for all configure tests. If a configure test failed due
# to -Werror this would just silently disable some features,
--
1.7.10.4
- [Qemu-devel] [PULL 02/15] q35: Fix typo in constant DEFUALT -> DEFAULT., (continued)
- [Qemu-devel] [PULL 02/15] q35: Fix typo in constant DEFUALT -> DEFAULT., Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 10/15] vscclient: remove unnecessary use of uninitialized variable, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 11/15] iov: avoid "orig_len may be used unitialized" warning, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 13/15] Makefile: Remove some more files when cleaning, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 09/15] trace-events: Clean up with scripts/cleanup-trace-events.pl again, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 06/15] MAINTAINERS: Add missing entry to filelist for TCI target, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 14/15] tcg-sparc: Fix parenthesis warning, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 12/15] target-i386: Fix segment cache dump, Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 08/15] tci: Fix qemu-alpha on 32 bit hosts (wrong assertions), Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 04/15] *-user: Fix typo in comment (ulocking -> unlocking), Michael Tokarev, 2013/09/14
- [Qemu-devel] [PULL 15/15] configure: Enable extra compiler warnings,
Michael Tokarev <=
- Re: [Qemu-devel] [PULL 00/15] trivial patches for 2013-09-14, Anthony Liguori, 2013/09/17