Merge pull request #63 from Garcia6l20/fix/cxx_17_compilation

fix c++17 compilation

Author: c-jimenez

3rdparty/CMakeLists.txt

@@ -1,6 +1,10 @@
 
 # Rapidjson is an header only library
 add_library(rapidjson INTERFACE)
+if("${CMAKE_CXX_STANDARD}" GREATER_EQUAL 17)
+    # use of std::iterator is deprecated since c++ 17
+    target_compile_definitions(rapidjson INTERFACE -DRAPIDJSON_NOMEMBERITERATORCLASS=1)
+endif()
 target_include_directories(rapidjson INTERFACE rapidjson/include)
 
 # Doctest is an header only on library

CMakeLists_native.txt

@@ -6,11 +6,11 @@
 set(WARNING_FLAGS "-Wall -Wextra -Werror")
 
 # C/C++ flags
-set(C_FLAGS "-std=c11")
-set(CXX_FLAGS "-std=c++17")
+set(CMAKE_C_STANDARD 11)
+set(CMAKE_CXX_STANDARD 17)
 
 # Set compiler flags
-set(CMAKE_C_FLAGS_DEBUG     "${C_FLAGS} ${WARNING_FLAGS} -O0 -g3 -ggdb3")
-set(CMAKE_CXX_FLAGS_DEBUG   "${CXX_FLAGS} ${WARNING_FLAGS} -O0 -g3 -ggdb3")
-set(CMAKE_C_FLAGS_RELEASE   "${C_FLAGS} ${WARNING_FLAGS} -O2 -DNDEBUG")
-set(CMAKE_CXX_FLAGS_RELEASE "${CXX_FLAGS} ${WARNING_FLAGS} -O2 -DNDEBUG")
+set(CMAKE_C_FLAGS_DEBUG     "${CMAKE_C_FLAGS_DEBUG_INIT} ${WARNING_FLAGS} -O0 -g3 -ggdb3")
+set(CMAKE_CXX_FLAGS_DEBUG   "${CMAKE_CXX_FLAGS_DEBUG_INIT} ${WARNING_FLAGS} -O0 -g3 -ggdb3")
+set(CMAKE_C_FLAGS_RELEASE   "${CMAKE_C_FLAGS_RELEASE_INIT} ${WARNING_FLAGS} -O2 -DNDEBUG")
+set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE_INIT} ${WARNING_FLAGS} -O2 -DNDEBUG")