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

Side by Side Diff: gyp/common_conditions.gypi

Issue 74193005: Add extra warnings to match what Android uses. (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Created 7 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | gyp/poppler.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # conditions used in both common.gypi and skia.gyp in chromium 1 # conditions used in both common.gypi and skia.gyp in chromium
2 # 2 #
3 { 3 {
4 'defines': [ 4 'defines': [
5 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)', 5 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)',
6 ], 6 ],
7 'conditions' : [ 7 'conditions' : [
8 [ 'skia_gpu == 1', 8 [ 'skia_gpu == 1',
9 { 9 {
10 'defines': [ 10 'defines': [
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 }, 142 },
143 }, 143 },
144 }], 144 }],
145 ], 145 ],
146 }, 146 },
147 ], 147 ],
148 148
149 # The following section is common to linux + derivatives and android 149 # The following section is common to linux + derivatives and android
150 [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "nacl", "chromeos", "android"]', 150 [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "nacl", "chromeos", "android"]',
151 { 151 {
152 'cflags': [
153 '-fno-exceptions',
154 '-fstrict-aliasing',
155
156 '-Wall',
157 '-Wextra',
158 '-Winit-self',
159 '-Wpointer-arith',
160
161 '-Wno-unused-parameter',
162 '-Wno-c++11-extensions',
163 ],
164 'cflags_cc': [
165 '-fno-rtti',
166 '-Wnon-virtual-dtor',
167 ],
152 'conditions': [ 168 'conditions': [
153 [ 'skia_warnings_as_errors', { 169 [ 'skia_warnings_as_errors', {
154 'cflags': [ 170 'cflags': [
155 '-Werror', 171 '-Werror',
156 ], 172 ],
157 }], 173 }],
158 [ 'skia_arch_type == "arm" and arm_thumb == 1', { 174 [ 'skia_arch_type == "arm" and arm_thumb == 1', {
159 'cflags': [ 175 'cflags': [
160 '-mthumb', 176 '-mthumb',
161 ], 177 ],
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
222 'cflags': ['-g'] 238 'cflags': ['-g']
223 }, 239 },
224 'Release': { 240 'Release': {
225 'cflags': [ 241 'cflags': [
226 '-O<(skia_release_optimization_level)', 242 '-O<(skia_release_optimization_level)',
227 '-g', 243 '-g',
228 ], 244 ],
229 'defines': [ 'NDEBUG' ], 245 'defines': [ 'NDEBUG' ],
230 }, 246 },
231 }, 247 },
232 'cflags': [
233 '-Wall',
234 '-Wextra',
235 # suppressions below here were added for clang
236 '-Wno-unused-parameter',
237 '-Wno-c++11-extensions',
238 ],
239 'conditions' : [ 248 'conditions' : [
240 [ 'skia_shared_lib', { 249 [ 'skia_shared_lib', {
241 'cflags': [ 250 'cflags': [
242 '-fPIC', 251 '-fPIC',
243 ], 252 ],
244 'defines': [ 253 'defines': [
245 'SKIA_DLL', 254 'SKIA_DLL',
246 'SKIA_IMPLEMENTATION=1', 255 'SKIA_IMPLEMENTATION=1',
247 ], 256 ],
248 }], 257 }],
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
467 'cflags': ['-O2'], 476 'cflags': ['-O2'],
468 'defines': [ 'NDEBUG' ], 477 'defines': [ 'NDEBUG' ],
469 }, 478 },
470 }, 479 },
471 'libraries': [ 480 'libraries': [
472 '-lstdc++', 481 '-lstdc++',
473 '-lm', 482 '-lm',
474 '-llog', 483 '-llog',
475 ], 484 ],
476 'cflags': [ 485 'cflags': [
477 '-Wall',
478 '-fno-exceptions',
479 '-fstrict-aliasing',
480 '-fuse-ld=gold', 486 '-fuse-ld=gold',
481 ], 487 ],
482 'cflags_cc': [
483 '-fno-rtti',
484 ],
485 'conditions': [ 488 'conditions': [
486 [ 'skia_shared_lib', { 489 [ 'skia_shared_lib', {
487 'cflags': [ 490 'cflags': [
488 '-fPIC', 491 '-fPIC',
489 ], 492 ],
490 'defines': [ 493 'defines': [
491 'SKIA_DLL', 494 'SKIA_DLL',
492 'SKIA_IMPLEMENTATION=1', 495 'SKIA_IMPLEMENTATION=1',
493 ], 496 ],
494 }], 497 }],
(...skipping 16 matching lines...) Expand all
511 'xcode_settings': { 514 'xcode_settings': {
512 'SYMROOT': '<(DEPTH)/xcodebuild', 515 'SYMROOT': '<(DEPTH)/xcodebuild',
513 }, 516 },
514 } 517 }
515 518
516 # Local Variables: 519 # Local Variables:
517 # tab-width:2 520 # tab-width:2
518 # indent-tabs-mode:nil 521 # indent-tabs-mode:nil
519 # End: 522 # End:
520 # vim: set expandtab tabstop=2 shiftwidth=2: 523 # vim: set expandtab tabstop=2 shiftwidth=2:
OLDNEW
« no previous file with comments | « no previous file | gyp/poppler.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698