- [enableval=no])
-if test "${enableval}" = yes; then
- gl_WARNING_CFLAGS([-Werror])
- AC_SUBST([WERROR_CFLAGS], [$WARNING_CFLAGS])
- WARNING_CFLAGS=
- gl_WARNING_CFLAGS([-Wall])
- gl_WARNING_CFLAGS([-Wextra -Wno-sign-compare])
- gl_WARNING_CFLAGS([-Wcast-align])
- gl_WARNING_CFLAGS([-Wcast-qual])
- gl_WARNING_CFLAGS([-Wformat])
- gl_WARNING_CFLAGS([-Wpointer-arith])
- gl_WARNING_CFLAGS([-Wwrite-strings])
- AC_SUBST([WARNING_CXXFLAGS], [$WARNING_CFLAGS])
- # The following warnings are not suitable for C++.
- gl_WARNING_CFLAGS([-Wbad-function-cast])
- gl_WARNING_CFLAGS([-Wmissing-declarations])
- gl_WARNING_CFLAGS([-Wmissing-prototypes])
- gl_WARNING_CFLAGS([-Wshadow])
- gl_WARNING_CFLAGS([-Wstrict-prototypes])
- AC_DEFINE([lint], 1, [Define to 1 if the compiler is checking for lint.])
+ [enable_gcc_warnings=no])
+AM_CONDITIONAL([ENABLE_GCC_WARNINGS], [test "$enable_gcc_warnings" = yes])
+if test "$enable_gcc_warnings" = yes; then
+ warn_common='-Wall-Wextra -Wno-sign-compare -Wcast-align -Wdocumentation
+ -Wformat -Wpointer-arith -Wwrite-strings'
+ warn_c='-Wbad-function-cast -Wshadow -Wstrict-prototypes'
+ warn_cxx='-Wnoexcept'
+ # Warnings for the test suite only.
+ #
+ # -fno-color-diagnostics: Clang's use of colors in the error
+ # messages is confusing the tests looking at the compiler's output
+ # (e.g., synclines.at).
+ warn_tests='-Wundef -pedantic -Wsign-compare -fno-color-diagnostics'
+
+ AC_LANG_PUSH([C])
+ # Clang supports many of GCC's -W options, but only issues warnings
+ # on the ones it does not recognize. In that case, gl_WARN_ADD
+ # thinks the option is supported, and unknown options are then added
+ # to CFLAGS. But then, when -Werror is added in the test suite for
+ # instance, the warning about the unknown option turns into an
+ # error.
+ #
+ # This should be addressed by gnulib's gl_WARN_ADD, but in the
+ # meanwhile, turn warnings about unknown options into errors in
+ # CFLAGS, and restore CFLAGS after the tests.
+ save_CFLAGS=$CFLAGS
+ gl_WARN_ADD([-Werror=unknown-warning-option], [CFLAGS])
+ for i in $warn_common $warn_c;
+ do
+ gl_WARN_ADD([$i], [WARN_CFLAGS])
+ done
+ gl_WARN_ADD([-Werror], [WERROR_CFLAGS])
+
+ # Warnings for the test suite, and maybe for bison if GCC is modern
+ # enough.
+ gl_WARN_ADD([-Wmissing-declarations], [WARN_CFLAGS_TEST])
+ gl_WARN_ADD([-Wmissing-prototypes], [WARN_CFLAGS_TEST])
+ test $lv_cv_gcc_pragma_push_works = yes &&
+ AS_VAR_APPEND([WARN_CFLAGS], [" $WARN_CFLAGS_TEST"])
+
+ # Warnings for the test suite only.
+ for i in $warn_tests;
+ do
+ gl_WARN_ADD([$i], [WARN_CFLAGS_TEST])
+ done
+ CFLAGS=$save_CFLAGS
+ AC_LANG_POP([C])
+
+ AC_LANG_PUSH([C++])
+ save_CXXFLAGS=$CXXFLAGS
+ gl_WARN_ADD([-Werror=unknown-warning-option], [CXXFLAGS])
+ for i in $warn_common $warn_cxx;
+ do
+ gl_WARN_ADD([$i], [WARN_CXXFLAGS])
+ done
+ gl_WARN_ADD([-Wzero-as-null-pointer-constant], [WARN_CXXFLAGS],
+ [AC_LANG_PROGRAM([], [nullptr])])
+ gl_WARN_ADD([-Werror], [WERROR_CXXFLAGS])
+ # Warnings for the test suite only.
+ for i in $warn_tests;
+ do
+ gl_WARN_ADD([$i], [WARN_CXXFLAGS_TEST])
+ done
+ # Clang++ 3.2+ reject C code generated by Flex.
+ gl_WARN_ADD([-Wno-null-conversion], [FLEX_SCANNER_CXXFLAGS])
+ # So does G++ 4.8...
+ gl_WARN_ADD([-Wno-sign-compare], [FLEX_SCANNER_CXXFLAGS])
+ # ... possiby in std=c++11 mode.
+ gl_WARN_ADD([-Wno-zero-as-null-pointer-constant], [FLEX_SCANNER_CXXFLAGS])
+ CXXFLAGS=$save_CXXFLAGS
+ AC_LANG_POP([C++])
projects / bison.git / blobdiff