"extra *compiler* warnings", not just "extra warnings".
Rename --enable-extra-warnings to --enable-extra-compiler-warnings, and have the message talking about "extra warnings" talk about "extra compiler warnings", to make it more uniform (the documentation for the --enable flag speaks of "additional compiler warnings") and to clarify that these are warnings from the compiler, not from *shark. Change-Id: Ic1a045670144f8d9eda2e3427142027e2a339156 Reviewed-on: https://code.wireshark.org/review/1230 Reviewed-by: Guy Harris <guy@alum.mit.edu>
This commit is contained in:
parent
cbdd515491
commit
4fdd528f89
|
@ -773,8 +773,8 @@ AC_WIRESHARK_CHECK_NON_CXX_WARNING_OPTION_ERROR
|
|||
# These are not enabled by default, because the warnings they produce
|
||||
# are very hard or impossible to eliminate.
|
||||
#
|
||||
AC_ARG_ENABLE(extra-warnings,
|
||||
AC_HELP_STRING( [--enable-extra-warnings],
|
||||
AC_ARG_ENABLE(extra-compiler-warnings,
|
||||
AC_HELP_STRING( [--enable-extra-compiler-warnings],
|
||||
[do additional compiler warnings @<:@default=no@:>@]),
|
||||
[
|
||||
wireshark_extra_flags=$enableval
|
||||
|
@ -1002,7 +1002,7 @@ warnings_as_errors_default="yes"
|
|||
AC_MSG_CHECKING(whether we should treat compiler warnings as errors)
|
||||
AC_ARG_ENABLE(warnings-as-errors,
|
||||
AC_HELP_STRING( [--enable-warnings-as-errors],
|
||||
[treat warnings as errors (only for GCC or clang) @<:@default=yes, unless extra warnings are enabled@:>@]),
|
||||
[treat warnings as errors (only for GCC or clang) @<:@default=yes, unless extra compiler warnings are enabled@:>@]),
|
||||
[
|
||||
if test "x$ac_supports_gcc_flags" = "xyes" -a "x$enableval" = "xyes"; then
|
||||
with_warnings_as_errors="yes"
|
||||
|
|
Loading…
Reference in New Issue