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

Side by Side Diff: build/standalone.gypi

Issue 792953005: Resolve all but two VC++ build warnings in pdfium. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Link to warning bugs, and globally suppress 4996 Created 5 years, 11 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 | « no previous file | core/src/fpdftext/fpdf_text_int.cpp » ('j') | pdfium.gyp » ('J')
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 'clang%': 0, 10 'clang%': 0,
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 ], 158 ],
159 'ldflags': [ 159 'ldflags': [
160 '-pthread', 160 '-pthread',
161 ], 161 ],
162 'msvs_cygwin_dirs': ['<(DEPTH)/v8/third_party/cygwin'], 162 'msvs_cygwin_dirs': ['<(DEPTH)/v8/third_party/cygwin'],
163 'msvs_configuration_attributes': { 163 'msvs_configuration_attributes': {
164 'OutputDirectory': '<(DEPTH)\\build\\$(ConfigurationName)', 164 'OutputDirectory': '<(DEPTH)\\build\\$(ConfigurationName)',
165 'IntermediateDirectory': '$(OutDir)\\obj\\$(ProjectName)', 165 'IntermediateDirectory': '$(OutDir)\\obj\\$(ProjectName)',
166 'CharacterSet': '1', 166 'CharacterSet': '1',
167 }, 167 },
168 'msvs_disabled_warnings': [4800, 4996],
168 'msvs_settings': { 169 'msvs_settings': {
169 'VCCLCompilerTool': { 170 'VCCLCompilerTool': {
170 'MinimalRebuild': 'false', 171 'MinimalRebuild': 'false',
171 'BufferSecurityCheck': 'true', 172 'BufferSecurityCheck': 'true',
172 'EnableFunctionLevelLinking': 'true', 173 'EnableFunctionLevelLinking': 'true',
173 'RuntimeTypeInfo': 'false', 174 'RuntimeTypeInfo': 'false',
174 'WarningLevel': '3', 175 'WarningLevel': '3',
175 'WarnAsError': 'false',
176 'DebugInformationFormat': '3', 176 'DebugInformationFormat': '3',
177 'Detect64BitPortabilityProblems': 'false', 177 'Detect64BitPortabilityProblems': 'false',
178 'conditions': [ 178 'conditions': [
179 [ 'msvs_multi_core_compile', { 179 [ 'msvs_multi_core_compile', {
180 'AdditionalOptions': ['/MP'], 180 'AdditionalOptions': ['/MP'],
181 }], 181 }],
182 ['component=="shared_library"', { 182 ['component=="shared_library"', {
183 'ExceptionHandling': '1', # /EHsc 183 'ExceptionHandling': '1', # /EHsc
184 }, { 184 }, {
185 'ExceptionHandling': '0', 185 'ExceptionHandling': '0',
186 }], 186 }],
187 ['target_arch=="x64"', {
188 # 64-bit warnings need to be resolved.
189 # https://code.google.com/p/pdfium/issues/detail?id=101
190 'WarnAsError': 'false',
191 }, {
192 'WarnAsError': 'true',
193 }],
187 ], 194 ],
188 }, 195 },
189 'VCLibrarianTool': { 196 'VCLibrarianTool': {
190 'AdditionalOptions': ['/ignore:4221'], 197 'AdditionalOptions': ['/ignore:4221'],
191 }, 198 },
192 'VCLinkerTool': { 199 'VCLinkerTool': {
193 'GenerateDebugInformation': 'true', 200 'GenerateDebugInformation': 'true',
194 'LinkIncremental': '1', 201 'LinkIncremental': '1',
195 # SubSystem values: 202 # SubSystem values:
196 # 0 == not set 203 # 0 == not set
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
256 }], 263 }],
257 ], # target_conditions 264 ], # target_conditions
258 }, # target_defaults 265 }, # target_defaults
259 }], # OS=="mac" 266 }], # OS=="mac"
260 ], 267 ],
261 'xcode_settings': { 268 'xcode_settings': {
262 # See comment in Chromium's common.gypi for why this is needed. 269 # See comment in Chromium's common.gypi for why this is needed.
263 'SYMROOT': '<(DEPTH)/xcodebuild', 270 'SYMROOT': '<(DEPTH)/xcodebuild',
264 } 271 }
265 } 272 }
OLDNEW
« no previous file with comments | « no previous file | core/src/fpdftext/fpdf_text_int.cpp » ('j') | pdfium.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698