Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(16)

Side by Side Diff: build_gyp/standalone.gypi

Issue 2011303003: Match GN and GYP compilation warning flags (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: address comments Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « BUILD.gn ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 PDFium Authors. All rights reserved. 1 # Copyright 2014 PDFium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # Definitions to be used when building stand-alone PDFium binaries. 5 # Definitions to be used when building stand-alone PDFium binaries.
6 6
7 { 7 {
8 'variables': { 8 'variables': {
9 'component%': 'static_library', 9 'component%': 'static_library',
10 'chromium_code%': 0, 10 'chromium_code%': 0,
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 'msvs_configuration_attributes': { 198 'msvs_configuration_attributes': {
199 'OutputDirectory': '<(DEPTH)\\build\\$(ConfigurationName)', 199 'OutputDirectory': '<(DEPTH)\\build\\$(ConfigurationName)',
200 'IntermediateDirectory': '$(OutDir)\\obj\\$(ProjectName)', 200 'IntermediateDirectory': '$(OutDir)\\obj\\$(ProjectName)',
201 'CharacterSet': '1', 201 'CharacterSet': '1',
202 }, 202 },
203 'msvs_disabled_warnings': [ 203 'msvs_disabled_warnings': [
204 # #### 204 # ####
205 # This section is PDFium specific. 205 # This section is PDFium specific.
206 # #### 206 # ####
207 207
208 # C4800: forcing value to bool 'true' or 'false' (performance warning) 208 # Now there is no PDFium specific flag. Add here if needed.
209 4800,
210 209
211 # #### 210 # ####
212 # This section should match Chromium's build/common.gypi. 211 # This section should match Chromium's build/common.gypi.
213 # #### 212 # ####
214 213
215 # C4091: 'typedef ': ignored on left of 'X' when no variable is 214 # C4091: 'typedef ': ignored on left of 'X' when no variable is
216 # declared. 215 # declared.
217 # This happens in a number of Windows headers. Dumb. 216 # This happens in a number of Windows headers. Dumb.
218 4091, 217 4091,
219 218
(...skipping 252 matching lines...) Expand 10 before | Expand all | Expand 10 after
472 }], 471 }],
473 ], 472 ],
474 'target_conditions': [ 473 'target_conditions': [
475 ['chromium_code==0', { 474 ['chromium_code==0', {
476 'variables': { 475 'variables': {
477 'clang_warning_flags': [ 476 'clang_warning_flags': [
478 # Suppress unused variable warnings from third-party libraries. 477 # Suppress unused variable warnings from third-party libraries.
479 '-Wno-unused-variable', 478 '-Wno-unused-variable',
480 ], 479 ],
481 }, 480 },
481 'msvs_disabled_warnings': [
482 # forcing value to bool 'true' or 'false' (performance warning)
483 4800,
484 ],
482 'msvs_settings': { 485 'msvs_settings': {
483 'VCCLCompilerTool': { 486 'VCCLCompilerTool': {
484 'WarningLevel': '3', 487 'WarningLevel': '3',
485 }, 488 },
486 }, 489 },
487 'conditions': [ 490 'conditions': [
488 ['OS!="win"', { 491 ['OS!="win"', {
489 # Remove -Wextra for third-party code. 492 # Remove -Wextra for third-party code.
490 'cflags!': [ '-Wextra' ], 493 'cflags!': [ '-Wextra' ],
491 }], 494 }],
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
530 ], 533 ],
531 }], # OS=="win" 534 }], # OS=="win"
532 ["use_goma==1", { 535 ["use_goma==1", {
533 'make_global_settings': [ 536 'make_global_settings': [
534 ['CC_wrapper', '<(gomadir)/gomacc'], 537 ['CC_wrapper', '<(gomadir)/gomacc'],
535 ['CXX_wrapper', '<(gomadir)/gomacc'], 538 ['CXX_wrapper', '<(gomadir)/gomacc'],
536 ], 539 ],
537 }], # use_goma==1 540 }], # use_goma==1
538 ], 541 ],
539 } 542 }
OLDNEW
« no previous file with comments | « BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698