summaryrefslogtreecommitdiff
path: root/build_gyp
diff options
context:
space:
mode:
authorweili <weili@chromium.org>2016-05-04 18:25:27 -0700
committerCommit bot <commit-bot@chromium.org>2016-05-04 18:25:27 -0700
commit6e1ae8614e7c81d8e6a6249ee73cfbf8abb0d099 (patch)
tree851d0ee896934ef0269e09db28b05f7f300c54cb /build_gyp
parentc777f486f84611d2fdd2d03af661b14955f9efb6 (diff)
downloadpdfium-6e1ae8614e7c81d8e6a6249ee73cfbf8abb0d099.tar.xz
Enable 'treating warnings as errors' for 64 bit Win builds
The reason to disable 'treating warnings as errors' earilier is due to c4267 warnings, which we already disabled uniformly in pdfium.gyp. Re-enable this now so that we can have the same level of compilation warnings/errors on 32 bit and 64 bit builds. Also fix two c4390 warnings on Windows release builds. Their warning messages are "empty controlled statement found; is this the intent?" Review-Url: https://codereview.chromium.org/1949303002
Diffstat (limited to 'build_gyp')
-rw-r--r--build_gyp/standalone.gypi8
1 files changed, 1 insertions, 7 deletions
diff --git a/build_gyp/standalone.gypi b/build_gyp/standalone.gypi
index 67ed204409..be18b5006c 100644
--- a/build_gyp/standalone.gypi
+++ b/build_gyp/standalone.gypi
@@ -300,6 +300,7 @@
'EnableFunctionLevelLinking': 'true',
'RuntimeTypeInfo': 'false',
'WarningLevel': '3',
+ 'WarnAsError': 'true',
'DebugInformationFormat': '3',
'Detect64BitPortabilityProblems': 'false',
'conditions': [
@@ -311,13 +312,6 @@
}, {
'ExceptionHandling': '0',
}],
- ['target_arch=="x64"', {
- # 64-bit warnings need to be resolved.
- # https://code.google.com/p/pdfium/issues/detail?id=101
- 'WarnAsError': 'false',
- }, {
- 'WarnAsError': 'true',
- }],
['clang==1', {
'AdditionalOptions': [
# Don't warn about unused function parameters.