Skip to content

Commit d4a49cf

Browse files
committed
Merge pull request open-source-parsers#466 from bkuhls/buildroot
CMakeLists.txt: Treat conversion warning as error only with JSONCPP_W…
2 parents 8bd4f94 + 17fc9b1 commit d4a49cf

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,11 @@ if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
107107
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Wconversion -Wshadow -Werror=conversion -Werror=sign-compare")
108108
elseif (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
109109
# using GCC
110-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Wconversion -Wshadow -Wextra -Werror=conversion")
110+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Wconversion -Wshadow -Wextra")
111111
# not yet ready for -Wsign-conversion
112112

113113
if (JSONCPP_WITH_STRICT_ISO AND NOT JSONCPP_WITH_WARNING_AS_ERROR)
114-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pedantic")
114+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror=conversion -pedantic")
115115
endif ()
116116
elseif (CMAKE_CXX_COMPILER_ID STREQUAL "Intel")
117117
# using Intel compiler

0 commit comments

Comments
 (0)