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

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

Issue 1215643003: Remove -Wno-unused-private-field clang warning suppression. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up GN Created 5 years, 5 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
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 # MAINTAINERS: 5 # MAINTAINERS:
6 # See the BUILD.gn file for more extensive comments and documentation. This 6 # See the BUILD.gn file for more extensive comments and documentation. This
7 # .gyp file exists only for compatibility with gyp. The variables defined below 7 # .gyp file exists only for compatibility with gyp. The variables defined below
8 # are used in equivalent targets in BUILD.GN. 8 # are used in equivalent targets in BUILD.GN.
9 9
10 { 10 {
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 'src/public', 153 'src/public',
154 ], 154 ],
155 'sources': [ 155 'sources': [
156 '<@(cld2_core_sources)', 156 '<@(cld2_core_sources)',
157 '<@(cld2_core_impl_sources)', 157 '<@(cld2_core_impl_sources)',
158 '<@(cld2_data_sources)', 158 '<@(cld2_data_sources)',
159 ], 159 ],
160 'conditions': [ 160 'conditions': [
161 ['OS=="win"', { 161 ['OS=="win"', {
162 'msvs_disabled_warnings': [4267], # size_t -> int conversion. 162 'msvs_disabled_warnings': [4267], # size_t -> int conversion.
163 'variables': {
164 'clang_warning_flags': [
Nico 2015/07/09 01:58:16 Are the fields windows-specific somehow? If not, m
benwells 2015/07/09 11:43:02 Ah, not windows specific. Moved out from this.
165 # cld_2 contains unused private fields.
166 '-Wno-unused-private-field',
167 ],
168 },
163 }], 169 }],
164 ['cld2_table_size==0', { 170 ['cld2_table_size==0', {
165 'sources+': ['<@(cld2_data_smallest_sources)'] 171 'sources+': ['<@(cld2_data_smallest_sources)']
166 }], 172 }],
167 ['cld2_table_size==2', { 173 ['cld2_table_size==2', {
168 'sources+': ['<@(cld2_data_largest_sources)'] 174 'sources+': ['<@(cld2_data_largest_sources)']
169 }], 175 }],
170 ], 176 ],
171 }, 177 },
172 178
(...skipping 12 matching lines...) Expand all
185 ], 191 ],
186 'sources': [ 192 'sources': [
187 '<@(cld2_core_sources)', 193 '<@(cld2_core_sources)',
188 '<@(cld2_core_impl_sources)', 194 '<@(cld2_core_impl_sources)',
189 '<@(cld2_dynamic_data_loader_sources)', 195 '<@(cld2_dynamic_data_loader_sources)',
190 ], 196 ],
191 'defines': ['CLD2_DYNAMIC_MODE'], 197 'defines': ['CLD2_DYNAMIC_MODE'],
192 }, 198 },
193 ], 199 ],
194 } 200 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698