For now, don't turn on -Wcast-align by default. There are at least some

bugs it points out that probably mean the code won't work on machines
that require alignment (e.g., SPARC machines), but we'll turn it on once
we fix them.  (clang is fussier than GCC about this.)

svn path=/trunk/; revision=50187
This commit is contained in:
Guy Harris 2013-06-27 10:35:06 +00:00
parent f3f156073b
commit b450609a55
2 changed files with 6 additions and 2 deletions

View File

@ -124,7 +124,6 @@ set(WIRESHARK_C_FLAGS
-Wendif-labels
-Wpointer-arith
-Warray-bounds
-Wcast-align
-Wformat-security
-Wshorten-64-to-32
-Wvla
@ -161,6 +160,7 @@ set(WIRESHARK_EXTRA_C_FLAGS
-fstrict-overflow -Wstrict-overflow=4
-Wunreachable-code
-Wunsafe-loop-optimizations
-Wcast-align
-Wcast-qual
-Wformat-security
-Wredundant-decls

View File

@ -561,6 +561,11 @@ AC_ARG_ENABLE(extra-gcc-checks,
#
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wmissing-prototypes)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wmissing-declarations)
#
# A bunch of "that might not work on SPARC" code blocks
# this one for now.
#
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wcast-align)
fi
],)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wall -W) # -W is now known as -Wextra
@ -570,7 +575,6 @@ AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wendif-labels)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wpointer-arith)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wno-pointer-sign, C)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Warray-bounds)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wcast-align)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wformat-security)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wold-style-definition, C)
AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wshorten-64-to-32)