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

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

Issue 429793006: Update several clang warnings after r287092. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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 | Annotate | Revision Log
« no previous file with comments | « third_party/harfbuzz-ng/harfbuzz.gyp ('k') | third_party/mesa/mesa_gensrc.gypi » ('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) 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 { 5 {
6 'includes': [ 6 'includes': [
7 '../../build/win_precompile.gypi', 7 '../../build/win_precompile.gypi',
8 ], 8 ],
9 'variables': { 9 'variables': {
10 'enabled_libjingle_device_manager%': 0, 10 'enabled_libjingle_device_manager%': 0,
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 'configurations': { 141 'configurations': {
142 'Debug': { 142 'Debug': {
143 'defines': [ 143 'defines': [
144 # TODO(sergeyu): Fix libjingle to use NDEBUG instead of 144 # TODO(sergeyu): Fix libjingle to use NDEBUG instead of
145 # _DEBUG and remove this define. See above as well. 145 # _DEBUG and remove this define. See above as well.
146 '_DEBUG', 146 '_DEBUG',
147 ], 147 ],
148 } 148 }
149 }, 149 },
150 }, 150 },
151 'variables': {
152 'clang_warning_flags_unset': [
153 # Don't warn about string->bool used in asserts.
154 '-Wstring-conversion',
155 ],
156 },
151 'conditions': [ 157 'conditions': [
152 ['"<(libpeer_target_type)"=="static_library"', { 158 ['"<(libpeer_target_type)"=="static_library"', {
153 'defines': [ 'LIBPEERCONNECTION_LIB=1' ], 159 'defines': [ 'LIBPEERCONNECTION_LIB=1' ],
154 }], 160 }],
155 ['use_openssl==1', { 161 ['use_openssl==1', {
156 'defines': [ 162 'defines': [
157 'SSL_USE_OPENSSL', 163 'SSL_USE_OPENSSL',
158 'HAVE_OPENSSL_SSL_H', 164 'HAVE_OPENSSL_SSL_H',
159 ], 165 ],
160 'dependencies': [ 166 'dependencies': [
(...skipping 25 matching lines...) Expand all
186 '../third_party/platformsdk_win7/files/Include', 192 '../third_party/platformsdk_win7/files/Include',
187 ], 193 ],
188 'conditions' : [ 194 'conditions' : [
189 ['target_arch == "ia32"', { 195 ['target_arch == "ia32"', {
190 'defines': [ 196 'defines': [
191 '_USE_32BIT_TIME_T', 197 '_USE_32BIT_TIME_T',
192 ], 198 ],
193 }], 199 }],
194 ], 200 ],
195 }], 201 }],
196 ['clang == 1', {
197 'xcode_settings': {
198 'WARNING_CFLAGS!': [
199 # Don't warn about string->bool used in asserts.
200 '-Wstring-conversion',
201 ],
202 },
203 'cflags!': [
204 '-Wstring-conversion',
205 ],
206 }],
207 ['OS=="linux"', { 202 ['OS=="linux"', {
208 'defines': [ 203 'defines': [
209 'LINUX', 204 'LINUX',
210 ], 205 ],
211 }], 206 }],
212 ['OS=="mac"', { 207 ['OS=="mac"', {
213 'defines': [ 208 'defines': [
214 'OSX', 209 'OSX',
215 ], 210 ],
216 }], 211 }],
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
648 # in this directory and not lib.target as will otherwise be 643 # in this directory and not lib.target as will otherwise be
649 # the case with make builds. 644 # the case with make builds.
650 'product_dir': '<(PRODUCT_DIR)/lib', 645 'product_dir': '<(PRODUCT_DIR)/lib',
651 }], 646 }],
652 ], 647 ],
653 }, # target libpeerconnection 648 }, # target libpeerconnection
654 ], 649 ],
655 }], 650 }],
656 ], 651 ],
657 } 652 }
OLDNEW
« no previous file with comments | « third_party/harfbuzz-ng/harfbuzz.gyp ('k') | third_party/mesa/mesa_gensrc.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698