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

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

Issue 12258039: enable -Wstring-conversion when compiling with clang (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: enable on mac, better fix in web_data_service Created 7 years, 10 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 | « sync/syncable/entry_kernel.cc ('k') | third_party/mesa/mesa.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 # 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 162 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 '../third_party/platformsdk_win7/files/Include', 173 '../third_party/platformsdk_win7/files/Include',
174 ], 174 ],
175 'conditions' : [ 175 'conditions' : [
176 ['target_arch == "ia32"', { 176 ['target_arch == "ia32"', {
177 'defines': [ 177 'defines': [
178 '_USE_32BIT_TIME_T', 178 '_USE_32BIT_TIME_T',
179 ], 179 ],
180 }], 180 }],
181 ], 181 ],
182 }], 182 }],
183 ['clang == 1', {
184 'xcode_settings': {
185 'WARNING_CFLAGS!': [
186 # Don't warn about string->bool used in asserts.
187 '-Wstring-conversion',
188 ],
189 },
190 'cflags!': [
191 '-Wstring-conversion',
192 ],
193 }],
183 ['OS=="linux"', { 194 ['OS=="linux"', {
184 'defines': [ 195 'defines': [
185 'LINUX', 196 'LINUX',
186 ], 197 ],
187 }], 198 }],
188 ['OS=="mac"', { 199 ['OS=="mac"', {
189 'defines': [ 200 'defines': [
190 'OSX', 201 'OSX',
191 ], 202 ],
192 }], 203 }],
(...skipping 620 matching lines...) Expand 10 before | Expand all | Expand 10 after
813 '<(DEPTH)/third_party/webrtc/system_wrappers/source/system_wrappers. gyp:system_wrappers', 824 '<(DEPTH)/third_party/webrtc/system_wrappers/source/system_wrappers. gyp:system_wrappers',
814 '<@(libjingle_peerconnection_additional_deps)', 825 '<@(libjingle_peerconnection_additional_deps)',
815 'libjingle', 826 'libjingle',
816 'libjingle_p2p', 827 'libjingle_p2p',
817 ], 828 ],
818 }, # target libjingle_peerconnection 829 }, # target libjingle_peerconnection
819 ], 830 ],
820 }], 831 }],
821 ], 832 ],
822 } 833 }
OLDNEW
« no previous file with comments | « sync/syncable/entry_kernel.cc ('k') | third_party/mesa/mesa.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698