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

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

Issue 2537293003: Remove warning suppression from VideoToolboxEncoder (Closed)
Patch Set: Created 4 years 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 | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2016 The WebRTC project authors. All Rights Reserved. 1 # Copyright 2016 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_mac) { 10 if (is_mac) {
(...skipping 12 matching lines...) Expand all
23 } 23 }
24 } 24 }
25 25
26 if (is_ios || is_mac) { 26 if (is_ios || is_mac) {
27 config("rtc_sdk_common_objc_config") { 27 config("rtc_sdk_common_objc_config") {
28 include_dirs = [ 28 include_dirs = [
29 "objc/Framework/Classes", 29 "objc/Framework/Classes",
30 "objc/Framework/Headers", 30 "objc/Framework/Headers",
31 ] 31 ]
32 } 32 }
33 config("webrtc_h264_video_toolbox_warnings_config") {
34 if (is_clang) {
35 # TODO(tkchin): Make webrtc_h264_video_toolbox compile with the standard s et
36 # of warnings.
37 # See https://bugs.chromium.org/p/webrtc/issues/detail?id=6307
38 cflags = [ "-Wno-thread-safety-analysis" ]
39 }
40 }
41 33
42 rtc_static_library("rtc_sdk_common_objc") { 34 rtc_static_library("rtc_sdk_common_objc") {
43 deps = [ 35 deps = [
44 "../base:rtc_base", 36 "../base:rtc_base",
45 ] 37 ]
46 configs += [ 38 configs += [
47 "..:common_objc", 39 "..:common_objc",
48 "//build/config/compiler:enable_arc", 40 "//build/config/compiler:enable_arc",
49 ] 41 ]
50 public_configs = [ ":rtc_sdk_common_objc_config" ] 42 public_configs = [ ":rtc_sdk_common_objc_config" ]
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
378 sources = [ 370 sources = [
379 "objc/Framework/Classes/h264_video_toolbox_decoder.cc", 371 "objc/Framework/Classes/h264_video_toolbox_decoder.cc",
380 "objc/Framework/Classes/h264_video_toolbox_decoder.h", 372 "objc/Framework/Classes/h264_video_toolbox_decoder.h",
381 "objc/Framework/Classes/h264_video_toolbox_encoder.h", 373 "objc/Framework/Classes/h264_video_toolbox_encoder.h",
382 "objc/Framework/Classes/h264_video_toolbox_encoder.mm", 374 "objc/Framework/Classes/h264_video_toolbox_encoder.mm",
383 "objc/Framework/Classes/h264_video_toolbox_nalu.cc", 375 "objc/Framework/Classes/h264_video_toolbox_nalu.cc",
384 "objc/Framework/Classes/h264_video_toolbox_nalu.h", 376 "objc/Framework/Classes/h264_video_toolbox_nalu.h",
385 ] 377 ]
386 378
387 configs += [ 379 configs += [
388 ":webrtc_h264_video_toolbox_warnings_config",
389 "//webrtc:common_objc", 380 "//webrtc:common_objc",
390 "//build/config/compiler:enable_arc", 381 "//build/config/compiler:enable_arc",
391 ] 382 ]
392 383
393 deps = [ 384 deps = [
394 ":rtc_sdk_common_objc", 385 ":rtc_sdk_common_objc",
395 ] 386 ]
396 387
397 libs = [ 388 libs = [
398 "CoreFoundation.framework", 389 "CoreFoundation.framework",
(...skipping 11 matching lines...) Expand all
410 deps += [ "$rtc_libyuv_dir" ] 401 deps += [ "$rtc_libyuv_dir" ]
411 public_deps = [ 402 public_deps = [
412 "$rtc_libyuv_dir", 403 "$rtc_libyuv_dir",
413 ] 404 ]
414 } else { 405 } else {
415 # Need to add a directory normally exported by libyuv. 406 # Need to add a directory normally exported by libyuv.
416 include_dirs = [ "$rtc_libyuv_dir/include" ] 407 include_dirs = [ "$rtc_libyuv_dir/include" ]
417 } 408 }
418 } 409 }
419 } 410 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698