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

Side by Side Diff: build/common.gypi

Issue 11361240: Add the -Wobjc-missing-property-synthesis flag. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: +mac Created 8 years, 1 month 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 | 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 (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium 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 # IMPORTANT: 5 # IMPORTANT:
6 # Please don't directly include this file if you are building via gyp_chromium, 6 # Please don't directly include this file if you are building via gyp_chromium,
7 # since gyp_chromium is automatically forcing its inclusion. 7 # since gyp_chromium is automatically forcing its inclusion.
8 { 8 {
9 # Variables expected to be overriden on the GYP command line (-D) or by 9 # Variables expected to be overriden on the GYP command line (-D) or by
10 # ~/.gyp/include.gypi. 10 # ~/.gyp/include.gypi.
(...skipping 3051 matching lines...) Expand 10 before | Expand all | Expand 10 after
3062 'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES', # -fvisibility=hidden 3062 'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES', # -fvisibility=hidden
3063 'GCC_THREADSAFE_STATICS': 'NO', # -fno-threadsafe-statics 3063 'GCC_THREADSAFE_STATICS': 'NO', # -fno-threadsafe-statics
3064 'GCC_TREAT_WARNINGS_AS_ERRORS': 'YES', # -Werror 3064 'GCC_TREAT_WARNINGS_AS_ERRORS': 'YES', # -Werror
3065 'GCC_VERSION': '4.2', 3065 'GCC_VERSION': '4.2',
3066 'GCC_WARN_ABOUT_MISSING_NEWLINE': 'YES', # -Wnewline-eof 3066 'GCC_WARN_ABOUT_MISSING_NEWLINE': 'YES', # -Wnewline-eof
3067 'USE_HEADERMAP': 'NO', 3067 'USE_HEADERMAP': 'NO',
3068 'WARNING_CFLAGS': [ 3068 'WARNING_CFLAGS': [
3069 '-Wall', 3069 '-Wall',
3070 '-Wendif-labels', 3070 '-Wendif-labels',
3071 '-Wextra', 3071 '-Wextra',
3072 # Warn if automatic synthesis is triggered.
3073 '-Wobjc-missing-property-synthesis',
Nico 2012/11/14 21:06:55 If you say 'CLANG_WARN_OBJC_MISSING_PROPERTY_SYNTH
Mark Mentovai 2012/11/15 00:06:52 There’s a symbolic name for this CLANG_WARN_OBJC_
3072 # Don't warn about unused function parameters. 3074 # Don't warn about unused function parameters.
3073 '-Wno-unused-parameter', 3075 '-Wno-unused-parameter',
3074 # Don't warn about the "struct foo f = {0};" initialization 3076 # Don't warn about the "struct foo f = {0};" initialization
3075 # pattern. 3077 # pattern.
3076 '-Wno-missing-field-initializers', 3078 '-Wno-missing-field-initializers',
3077 ], 3079 ],
3078 'conditions': [ 3080 'conditions': [
3079 ['chromium_mac_pch', {'GCC_PRECOMPILE_PREFIX_HEADER': 'YES'}, 3081 ['chromium_mac_pch', {'GCC_PRECOMPILE_PREFIX_HEADER': 'YES'},
3080 {'GCC_PRECOMPILE_PREFIX_HEADER': 'NO'} 3082 {'GCC_PRECOMPILE_PREFIX_HEADER': 'NO'}
3081 ], 3083 ],
(...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after
3308 ['OS=="ios"', { 3310 ['OS=="ios"', {
3309 'target_defaults': { 3311 'target_defaults': {
3310 'xcode_settings' : { 3312 'xcode_settings' : {
3311 'GCC_VERSION': 'com.apple.compilers.llvm.clang.1_0', 3313 'GCC_VERSION': 'com.apple.compilers.llvm.clang.1_0',
3312 3314
3313 # This next block is mostly common with the 'mac' section above, 3315 # This next block is mostly common with the 'mac' section above,
3314 # but keying off (or setting) 'clang' isn't valid for iOS as it 3316 # but keying off (or setting) 'clang' isn't valid for iOS as it
3315 # also seems to mean using the custom build of clang. 3317 # also seems to mean using the custom build of clang.
3316 3318
3317 # Don't use -Wc++0x-extensions, which Xcode 4 enables by default 3319 # Don't use -Wc++0x-extensions, which Xcode 4 enables by default
3318 # when buliding with clang. This warning is triggered when the 3320 # when building with clang. This warning is triggered when the
3319 # override keyword is used via the OVERRIDE macro from 3321 # override keyword is used via the OVERRIDE macro from
3320 # base/compiler_specific.h. 3322 # base/compiler_specific.h.
3321 'CLANG_WARN_CXX0X_EXTENSIONS': 'NO', 3323 'CLANG_WARN_CXX0X_EXTENSIONS': 'NO',
3322 'WARNING_CFLAGS': [ 3324 'WARNING_CFLAGS': [
3323 '-Wheader-hygiene', 3325 '-Wheader-hygiene',
3324 # Don't die on dtoa code that uses a char as an array index. 3326 # Don't die on dtoa code that uses a char as an array index.
3325 # This is required solely for base/third_party/dmg_fp/dtoa.cc. 3327 # This is required solely for base/third_party/dmg_fp/dtoa.cc.
3326 '-Wno-char-subscripts', 3328 '-Wno-char-subscripts',
3327 # Clang spots more unused functions. 3329 # Clang spots more unused functions.
3328 '-Wno-unused-function', 3330 '-Wno-unused-function',
3329 # See comments on this flag higher up in this file. 3331 # See comments on this flag higher up in this file.
3330 '-Wno-unnamed-type-template-args', 3332 '-Wno-unnamed-type-template-args',
3331 # This (rightyfully) complains about 'override', which we use 3333 # This (rightfully) complains about 'override', which we use
3332 # heavily. 3334 # heavily.
3333 '-Wno-c++11-extensions', 3335 '-Wno-c++11-extensions',
3334 ], 3336 ],
3335 }, 3337 },
3336 'target_conditions': [ 3338 'target_conditions': [
3337 ['_type=="executable"', { 3339 ['_type=="executable"', {
3338 'configurations': { 3340 'configurations': {
3339 'Release_Base': { 3341 'Release_Base': {
3340 'xcode_settings': { 3342 'xcode_settings': {
3341 'DEPLOYMENT_POSTPROCESSING': 'YES', 3343 'DEPLOYMENT_POSTPROCESSING': 'YES',
(...skipping 391 matching lines...) Expand 10 before | Expand all | Expand 10 after
3733 # settings in target dicts. SYMROOT is a special case, because many other 3735 # settings in target dicts. SYMROOT is a special case, because many other
3734 # Xcode variables depend on it, including variables such as 3736 # Xcode variables depend on it, including variables such as
3735 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 3737 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
3736 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 3738 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
3737 # files to appear (when present) in the UI as actual files and not red 3739 # files to appear (when present) in the UI as actual files and not red
3738 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 3740 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
3739 # and therefore SYMROOT, needs to be set at the project level. 3741 # and therefore SYMROOT, needs to be set at the project level.
3740 'SYMROOT': '<(DEPTH)/xcodebuild', 3742 'SYMROOT': '<(DEPTH)/xcodebuild',
3741 }, 3743 },
3742 } 3744 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698