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

Side by Side Diff: webrtc/api/BUILD.gn

Issue 2408133008: GN: Exclude suppressions of Chromium Clang warnings for Chromium builds. (Closed)
Patch Set: Created 4 years, 2 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 | « webrtc/BUILD.gn ('k') | webrtc/audio/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) 2015 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 import("../build/webrtc.gni") 9 import("../build/webrtc.gni")
10 if (is_android) { 10 if (is_android) {
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 "webrtcsdp.cc", 110 "webrtcsdp.cc",
111 "webrtcsdp.h", 111 "webrtcsdp.h",
112 "webrtcsession.cc", 112 "webrtcsession.cc",
113 "webrtcsession.h", 113 "webrtcsession.h",
114 "webrtcsessiondescriptionfactory.cc", 114 "webrtcsessiondescriptionfactory.cc",
115 "webrtcsessiondescriptionfactory.h", 115 "webrtcsessiondescriptionfactory.h",
116 ] 116 ]
117 117
118 configs += [ ":libjingle_peerconnection_warnings_config" ] 118 configs += [ ":libjingle_peerconnection_warnings_config" ]
119 119
120 if (is_clang) { 120 if (!build_with_chromium && is_clang) {
121 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163). 121 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
122 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 122 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
123 } 123 }
124 124
125 deps = [ 125 deps = [
126 ":call_api", 126 ":call_api",
127 ":rtc_stats_api", 127 ":rtc_stats_api",
128 "../call", 128 "../call",
129 "../media", 129 "../media",
130 "../pc", 130 "../pc",
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 "android/jni/native_handle_impl.h", 179 "android/jni/native_handle_impl.h",
180 "android/jni/peerconnection_jni.cc", 180 "android/jni/peerconnection_jni.cc",
181 "android/jni/surfacetexturehelper_jni.cc", 181 "android/jni/surfacetexturehelper_jni.cc",
182 "android/jni/surfacetexturehelper_jni.h", 182 "android/jni/surfacetexturehelper_jni.h",
183 "androidvideotracksource.cc", 183 "androidvideotracksource.cc",
184 "androidvideotracksource.h", 184 "androidvideotracksource.h",
185 ] 185 ]
186 186
187 configs += [ ":libjingle_peerconnection_jni_warnings_config" ] 187 configs += [ ":libjingle_peerconnection_jni_warnings_config" ]
188 188
189 if (is_clang) { 189 if (!build_with_chromium && is_clang) {
190 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163). 190 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
191 suppressed_configs += [ 191 suppressed_configs += [
192 "//build/config/clang:extra_warnings", 192 "//build/config/clang:extra_warnings",
193 "//build/config/clang:find_bad_constructs", 193 "//build/config/clang:find_bad_constructs",
194 ] 194 ]
195 } 195 }
196 196
197 # TODO(jschuh): Bug 1348: fix this warning. 197 # TODO(jschuh): Bug 1348: fix this warning.
198 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ] 198 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
199 199
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
375 "videocapturertracksource_unittest.cc", 375 "videocapturertracksource_unittest.cc",
376 "videotrack_unittest.cc", 376 "videotrack_unittest.cc",
377 "webrtcsdp_unittest.cc", 377 "webrtcsdp_unittest.cc",
378 "webrtcsession_unittest.cc", 378 "webrtcsession_unittest.cc",
379 ] 379 ]
380 380
381 defines = [ "HAVE_SCTP" ] 381 defines = [ "HAVE_SCTP" ]
382 382
383 configs += [ ":peerconnection_unittests_config" ] 383 configs += [ ":peerconnection_unittests_config" ]
384 384
385 if (is_clang) { 385 if (!build_with_chromium && is_clang) {
386 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163). 386 # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
387 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ] 387 suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
388 } 388 }
389 389
390 # TODO(jschuh): Bug 1348: fix this warning. 390 # TODO(jschuh): Bug 1348: fix this warning.
391 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ] 391 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
392 392
393 if (is_win) { 393 if (is_win) {
394 cflags = [ 394 cflags = [
395 "/wd4245", # conversion from int to size_t, signed/unsigned mismatch. 395 "/wd4245", # conversion from int to size_t, signed/unsigned mismatch.
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
465 465
466 shared_libraries = [ ":libjingle_peerconnection_so" ] 466 shared_libraries = [ ":libjingle_peerconnection_so" ]
467 } 467 }
468 468
469 android_resources("libjingle_peerconnection_android_unittest_resources") { 469 android_resources("libjingle_peerconnection_android_unittest_resources") {
470 resource_dirs = [ "androidtests/res" ] 470 resource_dirs = [ "androidtests/res" ]
471 custom_package = "org.webrtc" 471 custom_package = "org.webrtc"
472 } 472 }
473 } 473 }
474 } 474 }
OLDNEW
« no previous file with comments | « webrtc/BUILD.gn ('k') | webrtc/audio/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698