Skip to content

Commit 10478c5

Browse files
ChaseKnowldenslouken
authored andcommitted
Keep MSVC Flags Consistent across CMake runs
1 parent 62b9ac3 commit 10478c5

File tree

1 file changed

+23
-23
lines changed

1 file changed

+23
-23
lines changed

CMakeLists.txt

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -184,29 +184,6 @@ if(MSVC)
184184
set(SDL_RELOCATABLE_DEFAULT ON)
185185
endif()
186186

187-
if(MSVC)
188-
if(NOT SDL_LIBC)
189-
# Make sure /RTC1 is disabled, otherwise it will use functions from the CRT
190-
foreach(flag_var
191-
CMAKE_C_FLAGS CMAKE_C_FLAGS_DEBUG CMAKE_C_FLAGS_RELEASE
192-
CMAKE_C_FLAGS_MINSIZEREL CMAKE_C_FLAGS_RELWITHDEBINFO
193-
CMAKE_CXX_FLAGS CMAKE_CXX_FLAGS_DEBUG CMAKE_CXX_FLAGS_RELEASE
194-
CMAKE_CXX_FLAGS_MINSIZEREL CMAKE_CXX_FLAGS_RELWITHDEBINFO)
195-
string(REGEX REPLACE "/RTC(su|[1su])" "" ${flag_var} "${${flag_var}}")
196-
endforeach(flag_var)
197-
set(CMAKE_MSVC_RUNTIME_CHECKS "")
198-
endif()
199-
200-
if(MSVC_CLANG)
201-
# clang-cl treats /W4 as '-Wall -Wextra' -- we don't need -Wextra
202-
foreach(flag_var
203-
CMAKE_C_FLAGS CMAKE_C_FLAGS_DEBUG CMAKE_C_FLAGS_RELEASE
204-
CMAKE_C_FLAGS_MINSIZEREL CMAKE_C_FLAGS_RELWITHDEBINFO)
205-
string(REGEX REPLACE "/W4" "/W3" ${flag_var} "${${flag_var}}")
206-
endforeach(flag_var)
207-
endif()
208-
endif()
209-
210187
set(SDL_SHARED_DEFAULT ON)
211188
set(SDL_STATIC_DEFAULT ON)
212189

@@ -444,6 +421,29 @@ if(SDL_PRESEED)
444421
SDL_Preseed_CMakeCache()
445422
endif()
446423

424+
if(MSVC)
425+
if(NOT SDL_LIBC)
426+
# Make sure /RTC1 is disabled, otherwise it will use functions from the CRT
427+
foreach(flag_var
428+
CMAKE_C_FLAGS CMAKE_C_FLAGS_DEBUG CMAKE_C_FLAGS_RELEASE
429+
CMAKE_C_FLAGS_MINSIZEREL CMAKE_C_FLAGS_RELWITHDEBINFO
430+
CMAKE_CXX_FLAGS CMAKE_CXX_FLAGS_DEBUG CMAKE_CXX_FLAGS_RELEASE
431+
CMAKE_CXX_FLAGS_MINSIZEREL CMAKE_CXX_FLAGS_RELWITHDEBINFO)
432+
string(REGEX REPLACE "/RTC(su|[1su])" "" ${flag_var} "${${flag_var}}")
433+
endforeach(flag_var)
434+
set(CMAKE_MSVC_RUNTIME_CHECKS "")
435+
endif()
436+
437+
if(MSVC_CLANG)
438+
# clang-cl treats /W4 as '-Wall -Wextra' -- we don't need -Wextra
439+
foreach(flag_var
440+
CMAKE_C_FLAGS CMAKE_C_FLAGS_DEBUG CMAKE_C_FLAGS_RELEASE
441+
CMAKE_C_FLAGS_MINSIZEREL CMAKE_C_FLAGS_RELWITHDEBINFO)
442+
string(REGEX REPLACE "/W4" "/W3" ${flag_var} "${${flag_var}}")
443+
endforeach(flag_var)
444+
endif()
445+
endif()
446+
447447
if(SDL_SHARED)
448448
add_library(SDL3-shared SHARED)
449449
add_library(SDL3::SDL3-shared ALIAS SDL3-shared)

0 commit comments

Comments
 (0)