Commit 4a03913cfb9d37455dd51cc9d342cde8dbccd742

Sascha Cunz 2012-11-19T04:48:40

VS-Build: Disable /W4 parameter, as it is not picked up anyway My other PR revealed, that the /W4 parameter, we give to MSVC is ignored because cmake set CMAKE_C_FLAGS already to /W3 and we overwrite it. The command line tools gave me a D9025 warning for this on every file and looking into the project properties page on MSVC 2008 tells, that it has the warning level set to /W3. However, the warnings introduced by /W4 are far to useless for having them enabled. So just disable them.

1
2
3
4
5
6
7
8
9
10
11
12
13
diff --git a/CMakeLists.txt b/CMakeLists.txt
index bde872f..18f8533 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -90,7 +90,7 @@ IF (MSVC)
 
     STRING(REPLACE "/Zm1000" " " CMAKE_C_FLAGS "${CMAKE_C_FLAGS}")
 
-	SET(CMAKE_C_FLAGS "/W4 /MP /nologo /Zi ${CMAKE_C_FLAGS}")
+	SET(CMAKE_C_FLAGS "/MP /nologo /Zi ${CMAKE_C_FLAGS}")
 	IF (STDCALL)
 	  SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} /Gz")
 	ENDIF ()