Skip to content

Commit

Permalink
Remove extra /W4 warning level for VS in cmake
Browse files Browse the repository at this point in the history
/W3 is already the default and contains more than enough warnings. /W4 is just too noisy.
  • Loading branch information
scamille committed Oct 4, 2021
1 parent 86f55ff commit 170fb3e
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,7 @@ if (BUILD_GUI)
endif()

# enable warnings
if (MSVC)
target_compile_options(engine PUBLIC /W4)
else()
if (NOT MSVC)
target_compile_options(engine PUBLIC -Wall -Wextra)
endif()

Expand Down

0 comments on commit 170fb3e

Please sign in to comment.