Increase warning level to 4 on Visual Studio
This requires disabling most /W4 warnings so the patch doesn't get too large. I still fixed a few bugs so I didn't have to disable some more serious warnings. Most of these warnings are already enabled on gcc so it's mostly windows-specific code that is affected. BUG=none TEST=none Review URL: http://codereview.chromium.org/6902069 git-svn-id: http://src.chromium.org/svn/trunk/src/build@83840 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
This commit is contained in:
Родитель
74e0b0ded1
Коммит
48a5f1ec2e
13
common.gypi
13
common.gypi
|
@ -758,7 +758,8 @@
|
|||
'msvs_disabled_warnings': [4800],
|
||||
'msvs_settings': {
|
||||
'VCCLCompilerTool': {
|
||||
'WarnAsError': 'false',
|
||||
'WarningLevel': '3',
|
||||
'WarnAsError': 'false', # TODO(maruel): Enable it.
|
||||
'Detect64BitPortabilityProblems': 'false',
|
||||
},
|
||||
},
|
||||
|
@ -1515,14 +1516,20 @@
|
|||
'$(VSInstallDir)/VC/atlmfc/include',
|
||||
],
|
||||
'msvs_cygwin_dirs': ['<(DEPTH)/third_party/cygwin'],
|
||||
'msvs_disabled_warnings': [4351, 4396, 4503, 4819],
|
||||
'msvs_disabled_warnings': [4351, 4396, 4503, 4819,
|
||||
# TODO(maruel): These warnings are level 4. They will be slowly
|
||||
# removed as code is fixed.
|
||||
4100, 4121, 4125, 4127, 4130, 4131, 4189, 4201, 4238, 4244, 4245,
|
||||
4310, 4355, 4428, 4481, 4505, 4510, 4512, 4530, 4610, 4611, 4701,
|
||||
4702, 4706,
|
||||
],
|
||||
'msvs_settings': {
|
||||
'VCCLCompilerTool': {
|
||||
'MinimalRebuild': 'false',
|
||||
'BufferSecurityCheck': 'true',
|
||||
'EnableFunctionLevelLinking': 'true',
|
||||
'RuntimeTypeInfo': 'false',
|
||||
'WarningLevel': '3',
|
||||
'WarningLevel': '4',
|
||||
'WarnAsError': 'true',
|
||||
'DebugInformationFormat': '3',
|
||||
'conditions': [
|
||||
|
|
Загрузка…
Ссылка в новой задаче