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

Side by Side Diff: third_party/mach_override/mach_override.gyp

Issue 1557733002: Prepare for -Wall for third-party code, -Wextra for chromium_code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 | « third_party/mach_override/BUILD.gn ('k') | third_party/ocmock/BUILD.gn » ('j') | 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) 2011 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2011 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 { 5 {
6 'targets': [ 6 'targets': [
7 ], 7 ],
8 'conditions': [ 8 'conditions': [
9 ['OS=="mac" or (OS=="ios" and "<(GENERATOR)"=="ninja")', { 9 ['OS=="mac" or (OS=="ios" and "<(GENERATOR)"=="ninja")', {
10 'targets' : [ 10 'targets' : [
(...skipping 29 matching lines...) Expand all
40 # Rather than patching a file that's not relevant, disable it. 40 # Rather than patching a file that's not relevant, disable it.
41 'libudis86/syn-att.c', 41 'libudis86/syn-att.c',
42 'libudis86/syn-intel.c', 42 'libudis86/syn-intel.c',
43 'libudis86/syn.c', 43 'libudis86/syn.c',
44 ], 44 ],
45 'variables': { 45 'variables': {
46 'clang_warning_flags_unset': [ 46 'clang_warning_flags_unset': [
47 # For UD_ASSERT(!"message"); 47 # For UD_ASSERT(!"message");
48 '-Wstring-conversion', 48 '-Wstring-conversion',
49 ], 49 ],
50 'clang_warning_flags': [
51 # syn.c contains a switch with an assert(false) in a default:
52 # block. In release builds, the function is missing a return.
53 '-Wno-return-type',
54 # Fires once in decode.c.
55 '-Wno-sometimes-uninitialized',
56 ],
50 }, 57 },
51 }, 58 },
52 { 59 {
53 'target_name' : 'mach_override', 60 'target_name' : 'mach_override',
54 'type': 'static_library', 61 'type': 'static_library',
55 'toolsets': ['host', 'target'], 62 'toolsets': ['host', 'target'],
56 'dependencies': [ 63 'dependencies': [
57 'libudis86', 64 'libudis86',
58 ], 65 ],
59 'sources': [ 66 'sources': [
60 'mach_override.c', 67 'mach_override.c',
61 'mach_override.h', 68 'mach_override.h',
62 ], 69 ],
63 'include_dirs': [ 70 'include_dirs': [
64 '../..', 71 '../..',
65 ], 72 ],
66 }, 73 },
67 ], 74 ],
68 }], 75 }],
69 ], 76 ],
70 } 77 }
OLDNEW
« no previous file with comments | « third_party/mach_override/BUILD.gn ('k') | third_party/ocmock/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698