[CEA 40693] COTECH 156.5: Check usage of C++14 as default standard for SALOME modules

[tools/configuration.git] / cmake / SalomeSetupPlatform.cmake

diff --git a/cmake/SalomeSetupPlatform.cmake b/cmake/SalomeSetupPlatform.cmake
index 3d7ef60cdaa37b79cd2f46857be973fee61344ff..892d1836a732829034b88b4be64a45047d207ca1 100644 (file)
--- a/cmake/SalomeSetupPlatform.cmake
+++ b/cmake/SalomeSetupPlatform.cmake
@@ -1,4 +1,4 @@
-# Copyright (C) 2007-2016  CEA/DEN, EDF R&D, OPEN CASCADE
+# Copyright (C) 2007-2024  CEA, EDF, OPEN CASCADE
 #
 # This library is free software; you can redistribute it and/or
 # modify it under the terms of the GNU Lesser General Public
@@ -48,16 +48,6 @@ ELSE()
   SET(PYLOGLEVEL WARNING)
 ENDIF()
 
-## Treat all warnings as errors
-IF(NOT (WIN32 OR APPLE))
-  OPTION(SALOME_DEBUG_WARNINGS "Treat warnings as errors" OFF)
-  MARK_AS_ADVANCED(SALOME_DEBUG_WARNINGS)
-  IF(SALOME_DEBUG_WARNINGS)
-    SET(CMAKE_C_FLAGS "-Werror")
-    SET(CMAKE_CXX_FLAGS "-Werror")
-  ENDIF()
-ENDIF()
-
 IF(WIN32)
   ## Windows specific:  
   ADD_DEFINITIONS(-D_CRT_SECURE_NO_WARNINGS)  # To disable windows warnings for strcpy, fopen, ...
@@ -65,6 +55,7 @@ IF(WIN32)
   ADD_DEFINITIONS(-DWNT -DWIN32)
   ADD_DEFINITIONS(-D_WIN32_WINNT=0x0500)      # Windows 2000 or later API is required
   ADD_DEFINITIONS(-DPPRO_NT)                  # For medfile
+  ADD_DEFINITIONS(-DNOMINMAX)
 
   SET(PLATFORM_LIBS Ws2_32.lib)
   LIST(APPEND PLATFORM_LIBS Userenv.lib)      # At least for GEOM suit
@@ -74,6 +65,8 @@ IF(WIN32)
   ELSE()
     SET(SIZE_OF_LONG ${CMAKE_SIZEOF_VOID_P})  # Set sizeof(long) the same as size of pointers
   ENDIF()
+  ADD_DEFINITIONS(-DUNICODE)                  # Unicode 
+  ADD_DEFINITIONS(-D_UNICODE)
 ELSE()
   ## Linux specific:
   SET(PLATFORM_LIBS dl)                       # Dynamic loading (dlopen, dlsym)
@@ -114,17 +107,51 @@ IF(COMPILER_SUPPORTS_NO_OPERATOR_NAMES)
   SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fno-operator-names")
 ENDIF()
 
-IF(NOT NO_CXX11_SUPPORT)
-  # C++11 support
-  CHECK_CXX_COMPILER_FLAG("-std=c++11" COMPILER_SUPPORTS_CXX11)
-  CHECK_CXX_COMPILER_FLAG("-std=c++0x" COMPILER_SUPPORTS_CXX0X)
-  IF(COMPILER_SUPPORTS_CXX11)
-    MESSAGE(STATUS "Enable C++11 support")
-    SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
-  ELSEIF(COMPILER_SUPPORTS_CXX0X)
-    MESSAGE(STATUS "Enable C++0x support")
-    SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++0x")
-  ELSE()
-    MESSAGE(WARNING "Compiler ${CMAKE_CXX_COMPILER} has no C++11 support.")
+# Option: C++ standard to use
+# Note: CMake by default enables C++ extensions; they can be disabled by additionally
+#       specifying -DCMAKE_CXX_EXTENSIONS=OFF
+SET(SALOME_CXX_STANDARD 14 CACHE STRING "C++ standard to use")
+SET(_supported_standards 11 14 17 20)
+SET(_standard_ok FALSE)
+FOREACH(_standard ${_supported_standards})
+  IF(SALOME_CXX_STANDARD STREQUAL _standard)
+    SET(_standard_ok TRUE)
+    BREAK()
+  ENDIF()
+ENDFOREACH()
+IF(NOT _standard_ok)
+  MESSAGE(FATAL_ERROR "Unsupported C++ standard: ${SALOME_CXX_STANDARD}; allowed values: ${_supported_standards}")
+ENDIF()
+UNSET(_supported_standards)
+UNSET(_standard)
+UNSET(_standard_ok)
+MESSAGE(STATUS "Setting C++ standard to ${SALOME_CXX_STANDARD}")
+SET(CMAKE_CXX_STANDARD ${SALOME_CXX_STANDARD})
+SET(CMAKE_CXX_STANDARD_REQUIRED ON)
+
+# Fight warnings
+IF(NOT APPLE)
+  OPTION(SALOME_DEBUG_WARNINGS "Report more warnings" OFF)
+  OPTION(SALOME_TREAT_WARNINGS_AS_ERRORS "Treat warnings as errors" OFF)
+  # Report more warnings
+  MARK_AS_ADVANCED(SALOME_DEBUG_WARNINGS SALOME_TREAT_WARNINGS_AS_ERRORS)
+  IF(SALOME_DEBUG_WARNINGS)
+    IF(WIN32)
+      SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} /W4")
+      SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /W4")
+    ELSE()
+      SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wall -Wextra -Wpedantic")
+      SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wpedantic")
+    ENDIF()
+  ENDIF()
+  ## Treat all warnings as errors
+  IF(SALOME_TREAT_WARNINGS_AS_ERRORS)
+    IF(WIN32)
+      SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} /WX")
+      SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /WX")
+    ELSE()
+      SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Werror")
+      SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror")
+    ENDIF()
   ENDIF()
 ENDIF()