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

Side by Side Diff: webrtc/modules/audio_coding/BUILD.gn

Issue 2291503002: Fix Chromium clang plugin warnings (Closed)
Patch Set: Created 4 years, 3 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/common.gyp ('k') | webrtc/modules/audio_coding/acm2/acm_send_test_oldapi.h » ('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) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 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 import("audio_coding.gni") 10 import("audio_coding.gni")
(...skipping 897 matching lines...) Expand 10 before | Expand all | Expand 10 after
908 source_set("acm_send_test") { 908 source_set("acm_send_test") {
909 testonly = true 909 testonly = true
910 sources = [ 910 sources = [
911 "acm2/acm_send_test_oldapi.cc", 911 "acm2/acm_send_test_oldapi.cc",
912 "acm2/acm_send_test_oldapi.h", 912 "acm2/acm_send_test_oldapi.h",
913 ] 913 ]
914 914
915 configs += [ "../..:common_config" ] 915 configs += [ "../..:common_config" ]
916 public_configs = [ "../..:common_inherited_config" ] 916 public_configs = [ "../..:common_inherited_config" ]
917 917
918 if (is_clang) {
919 # Suppress warnings from the Chromium Clang plugins (bugs.webrtc.org/163).
920 configs -= [ "//build/config/clang:find_bad_constructs" ]
921 }
922
923 defines = audio_coding_defines 918 defines = audio_coding_defines
924 919
925 deps = audio_coding_deps + [ 920 deps = audio_coding_deps + [
926 ":audio_coding", 921 ":audio_coding",
927 ":neteq_unittest_tools", 922 ":neteq_unittest_tools",
928 "//testing/gtest", 923 "//testing/gtest",
929 ] 924 ]
930 } 925 }
931 926
932 executable("delay_test") { 927 executable("delay_test") {
933 testonly = true 928 testonly = true
934 sources = [ 929 sources = [
935 "test/Channel.cc", 930 "test/Channel.cc",
936 "test/PCMFile.cc", 931 "test/PCMFile.cc",
937 "test/delay_test.cc", 932 "test/delay_test.cc",
938 "test/utility.cc", 933 "test/utility.cc",
939 ] 934 ]
940 935
941 configs += [ "../..:common_config" ] 936 configs += [ "../..:common_config" ]
942 public_configs = [ "../..:common_inherited_config" ] 937 public_configs = [ "../..:common_inherited_config" ]
943 938
944 if (is_clang) {
945 # Suppress warnings from Chrome's Clang plugins.
946 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
947 configs -= [ "//build/config/clang:find_bad_constructs" ]
948 }
949
950 deps = [ 939 deps = [
951 ":audio_coding", 940 ":audio_coding",
952 "../../:webrtc_common", 941 "../../:webrtc_common",
953 "../../system_wrappers", 942 "../../system_wrappers",
954 "../../system_wrappers:system_wrappers_default", 943 "../../system_wrappers:system_wrappers_default",
955 "../../test:test_support", 944 "../../test:test_support",
956 "../rtp_rtcp", 945 "../rtp_rtcp",
957 "//build/config/sanitizers:deps", 946 "//build/config/sanitizers:deps",
958 "//testing/gtest", 947 "//testing/gtest",
959 "//third_party/gflags:gflags", 948 "//third_party/gflags:gflags",
(...skipping 651 matching lines...) Expand 10 before | Expand all | Expand 10 after
1611 "//testing/gtest", 1600 "//testing/gtest",
1612 ] 1601 ]
1613 1602
1614 if (is_clang) { 1603 if (is_clang) {
1615 # Suppress warnings from Chrome's Clang plugins. 1604 # Suppress warnings from Chrome's Clang plugins.
1616 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details. 1605 # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
1617 configs -= [ "//build/config/clang:find_bad_constructs" ] 1606 configs -= [ "//build/config/clang:find_bad_constructs" ]
1618 } 1607 }
1619 } 1608 }
1620 } 1609 }
OLDNEW
« no previous file with comments | « webrtc/common.gyp ('k') | webrtc/modules/audio_coding/acm2/acm_send_test_oldapi.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698