Rename ENABLE_EXTRA_COMPILER_CHECKS to ENABLE_EXTRA_COMPILER_WARNINGS.
This matches the current name of the --enable-extra-compiler-warnings option in autotools. Fix the documentation of the option to match. (Note that "compiler" won't necessarily always be GCC or Clang, and won't necessarily always use -W for warning options, so speaking of them as "-W checks" isn't future-proof.) Change-Id: I2e142532e78be3c8051f1e738b3109a83e7d10dc Reviewed-on: https://code.wireshark.org/review/1231 Reviewed-by: Guy Harris <guy@alum.mit.edu>
This commit is contained in:
parent
4fdd528f89
commit
fde8ef0713
|
@ -304,7 +304,7 @@ else()
|
|||
)
|
||||
endif()
|
||||
|
||||
if(ENABLE_EXTRA_COMPILER_CHECKS) # This overrides -Werror
|
||||
if(ENABLE_EXTRA_COMPILER_WARNINGS) # This overrides -Werror
|
||||
set(WIRESHARK_C_FLAGS ${WIRESHARK_C_FLAGS} ${WIRESHARK_EXTRA_COMPILER_C_FLAGS})
|
||||
set(WIRESHARK_C_ONLY_FLAGS ${WIRESHARK_C_ONLY_FLAGS} ${WIRESHARK_EXTRA_COMPILER_C_ONLY_FLAGS})
|
||||
set(WIRESHARK_CPP_ONLY_FLAGS ${WIRESHARK_CPP_ONLY_FLAGS} ${WIRESHARK_EXTRA_COMPILER_CPP_ONLY_FLAGS})
|
||||
|
@ -323,7 +323,7 @@ set( CPP_FLAG_TESTS ${WIRESHARK_C_FLAGS} ${WIRESHARK_CPP_ONLY_FLAGS} )
|
|||
include(CheckCCompilerFlag)
|
||||
include(CheckCXXCompilerFlag)
|
||||
|
||||
if(NOT DISABLE_WERROR AND NOT ENABLE_EXTRA_COMPILER_CHECKS)
|
||||
if(NOT DISABLE_WERROR AND NOT ENABLE_EXTRA_COMPILER_WARNINGS)
|
||||
check_c_compiler_flag(-Werror=unknown-warning-option WERR_UNKNOWN)
|
||||
check_c_compiler_flag(-Werror WERROR)
|
||||
else()
|
||||
|
|
|
@ -16,8 +16,8 @@ option(BUILD_dftest "Build dftest" ON)
|
|||
option(AUTOGEN_dcerpc "Autogenerate DCE RPC dissectors" OFF)
|
||||
option(AUTOGEN_pidl "Autogenerate pidl dissectors" OFF)
|
||||
|
||||
option(DISABLE_WERROR "Do not treat Warnings as errors" OFF)
|
||||
option(ENABLE_EXTRA_COMPILER_CHECKS "Do additional -W checks (disables -Werror)" OFF)
|
||||
option(DISABLE_WERROR "Do not treat warnings as errors" OFF)
|
||||
option(ENABLE_EXTRA_COMPILER_WARNINGS "Do additional compiler warnings (disables -Werror)" OFF)
|
||||
option(ENABLE_GTK3 "Use GTK3 instead of GTK2 to build wireshark" ON)
|
||||
option(ENABLE_QT5 "Use Qt5 instead of Qt4 to build qtshark" ON)
|
||||
option(ENABLE_PCAP "Enable libpcap support (required for capturing)" ON)
|
||||
|
|
Loading…
Reference in New Issue