mirror of
https://gitlab.uni-freiburg.de/opensourcevdi/spice
synced 2026-01-14 03:35:52 +00:00
build: Cleanup disabled warnings a bit
Some C-specific warnings were turned off as the spice module used to ship a C++ spice client. It's long gone, so we can reenable these warnings. There were warnings disabled for tunnel support and CEGUI usage which have both been removed too. -Wno-format-nonliteral was disabled twice. Acked-by: Frediano Ziglio <fziglio@redhat.com>
This commit is contained in:
parent
81dbb18ccd
commit
f611228bd0
@ -71,18 +71,6 @@ AC_DEFUN([SPICE_COMPILE_WARNINGS],[
|
||||
# decl mess with gtk/generated_*.c
|
||||
dontwarn="$dontwarn -Wmissing-declarations"
|
||||
|
||||
# Stuff that C++ won't allow. Turn them back on later
|
||||
dontwarn="$dontwarn -Waggregate-return"
|
||||
dontwarn="$dontwarn -Wstrict-prototypes"
|
||||
dontwarn="$dontwarn -Wold-style-definition"
|
||||
dontwarn="$dontwarn -Wnested-externs"
|
||||
dontwarn="$dontwarn -Wformat-zero-length"
|
||||
dontwarn="$dontwarn -Wpointer-to-int-cast"
|
||||
dontwarn="$dontwarn -Wjump-misses-init"
|
||||
|
||||
# Issues in x11/platform_utils.cpp prevent us turning this on
|
||||
dontwarn="$dontwarn -Wmissing-format-attribute"
|
||||
|
||||
# Get all possible GCC warnings
|
||||
gl_MANYWARN_ALL_GCC([maybewarn])
|
||||
|
||||
@ -103,15 +91,6 @@ AC_DEFUN([SPICE_COMPILE_WARNINGS],[
|
||||
# We can't enable this due to horrible spice_usb_device_get_description
|
||||
# signature
|
||||
gl_WARN_ADD([-Wno-format-nonliteral])
|
||||
# CEGui headers cause violation of this
|
||||
gl_WARN_ADD([-Wno-ignored-qualifiers])
|
||||
|
||||
|
||||
|
||||
|
||||
# GNULIB turns on -Wformat=2 which implies -Wformat-nonliteral,
|
||||
# so we need to manually re-exclude it.
|
||||
gl_WARN_ADD([-Wno-format-nonliteral])
|
||||
|
||||
# This should be < 1024 really. pixman_utils is the blackspot
|
||||
# preventing lower usage
|
||||
@ -158,18 +137,6 @@ AC_DEFUN([SPICE_COMPILE_WARNINGS],[
|
||||
WARN_CXXFLAGS=$WARN_CFLAGS
|
||||
AC_SUBST([WARN_CXXFLAGS])
|
||||
|
||||
# These are C-only warnings
|
||||
if test "x$enable_tunnel" != "xyes"; then
|
||||
# Slirp causes pain :-(
|
||||
gl_WARN_ADD([-Waggregate-return])
|
||||
fi
|
||||
gl_WARN_ADD([-Wstrict-prototypes])
|
||||
gl_WARN_ADD([-Wold-style-definition])
|
||||
gl_WARN_ADD([-Wnested-externs])
|
||||
gl_WARN_ADD([-Wformat-zero-length])
|
||||
gl_WARN_ADD([-Wpointer-to-int-cast])
|
||||
gl_WARN_ADD([-Wjump-misses-init])
|
||||
|
||||
WARN_LDFLAGS=$WARN_CFLAGS
|
||||
AC_SUBST([WARN_CFLAGS])
|
||||
AC_SUBST([WARN_LDFLAGS])
|
||||
|
||||
Loading…
Reference in New Issue
Block a user