Skip to content

Commit 8c46511

Browse files
committed
Remove -Wextra and -Wall instead of disabling individual warnings.
1 parent 0ae4ad4 commit 8c46511

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

CMakeLists.txt

+4-10
Original file line numberDiff line numberDiff line change
@@ -46,21 +46,15 @@ include_directories(${CMAKE_CURRENT_SOURCE_DIR})
4646

4747
include(CheckCXXCompilerFlag)
4848
if (GECODE_DISABLE_WARNINGS)
49-
if (CMAKE_COMPILER_IS_GNUCXX)
50-
add_definitions(-Wno-overloaded-virtual)
51-
add_definitions(-Wno-switch)
52-
add_definitions(-Wno-unused-parameter)
53-
check_cxx_compiler_flag(
54-
-Wno-unused-but-set-variable HAVE_WNO_UNUSED_BUT_SET_VARIABLE_FLAG)
55-
if (HAVE_WNO_UNUSED_BUT_SET_VARIABLE_FLAG)
56-
add_definitions(-Wno-unused-but-set-variable)
57-
endif ()
58-
elseif (MSVC)
49+
if (MSVC)
5950
foreach (flag_var
6051
CMAKE_CXX_FLAGS CMAKE_CXX_FLAGS_DEBUG CMAKE_CXX_FLAGS_RELEASE
6152
CMAKE_CXX_FLAGS_MINSIZEREL CMAKE_CXX_FLAGS_RELWITHDEBINFO)
6253
string(REPLACE "/W3" "/w" ${flag_var} "${${flag_var}}")
6354
endforeach()
55+
else ()
56+
string(REPLACE "-Wextra" "" CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS}")
57+
string(REPLACE "-Wall" "" CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS}")
6458
endif ()
6559
endif ()
6660

0 commit comments

Comments
 (0)