compileroptions.cmake: added note about disabling -Wswitch-default

danmar · Oct 17, 2024 · a7c35a6 · a7c35a6
1 parent 7b77dcf
commit a7c35a6
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/cmake/compileroptions.cmake b/cmake/compileroptions.cmake
@@ -102,7 +102,6 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
     add_compile_options_safe(-Wno-date-time)
     add_compile_options(-Wno-disabled-macro-expansion)
     add_compile_options_safe(-Wno-bitwise-instead-of-logical)
-    add_compile_options_safe(-Wno-switch-default)
 
     # these cannot be fixed properly without adopting later C++ standards
     add_compile_options_safe(-Wno-unsafe-buffer-usage)
@@ -119,6 +118,9 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
     # only needs to be addressed to work around issues in older compilers
     add_compile_options_safe(-Wno-return-std-move-in-c++11)
 
+    # this is reported even when it is unnecessary i.e. -Wswitch-enum warnings have been mitigated
+    add_compile_options_safe(-Wno-switch-default)
+
     # warnings we are currently not interested in
     add_compile_options(-Wno-four-char-constants)
     add_compile_options(-Wno-weak-vtables)