diff options
author | weili <weili@chromium.org> | 2016-05-26 11:53:12 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-05-26 11:53:12 -0700 |
commit | 60607c3baaf52574cab748a97c213e447c9108c0 (patch) | |
tree | fb5ec2b56e97ef5019e4061d5cfd19eec7bfa0d6 /xfa.gyp | |
parent | fd6cfad310187631554613c375ec1b4c5ef887a3 (diff) | |
download | pdfium-60607c3baaf52574cab748a97c213e447c9108c0.tar.xz |
Enable chromium_code standard for XFA code
Clean up the left code which causes warnings. Enable using chromium_code
for XFA compilation, also re-enable all the msvs warning flags except
4267 (same as the main pdfium code).
BUG=pdfium:29,pdfium:475
Review-Url: https://codereview.chromium.org/2009813004
Diffstat (limited to 'xfa.gyp')
-rw-r--r-- | xfa.gyp | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -1,5 +1,6 @@ { "variables": { + "chromium_code": 1, "pdf_enable_v8%": 1, }, "target_defaults": { @@ -7,9 +8,7 @@ "PDF_ENABLE_XFA", ], 'msvs_disabled_warnings': [ - 4005, 4018, 4146, 4333, 4345, 4267, - # TODO(thestig): Fix all instances, remove this, pdfium:29 - 4245, 4310, 4389, 4701, 4702, 4706, 4800, + 4267, ], }, "targets":[ @@ -699,6 +698,9 @@ "conditions": [ ["clang==1" , { }], + ["os_posix==1 and clang==0", { # When GCC + 'cflags': [ '-Wno-error=strict-overflow' ], + }], ["pdf_enable_v8==1", { 'dependencies': [ '<(DEPTH)/v8/src/v8.gyp:v8', |