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

Unified Diff: webrtc/common_audio/BUILD.gn

Issue 2719743002: Enable GN check for webrtc/common_audio (Closed)
Patch Set: Rebased Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « .gn ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/common_audio/BUILD.gn
diff --git a/webrtc/common_audio/BUILD.gn b/webrtc/common_audio/BUILD.gn
index a3cf046bb46ddb4dcdb655e7e3d83ef6c305d522..6162793a6d38b54711e976dc1b2c968fa55ab5d1 100644
--- a/webrtc/common_audio/BUILD.gn
+++ b/webrtc/common_audio/BUILD.gn
@@ -46,7 +46,6 @@ rtc_static_library("common_audio") {
"resampler/push_sinc_resampler.h",
"resampler/resampler.cc",
"resampler/sinc_resampler.cc",
- "resampler/sinc_resampler.h",
"smoothing_filter.cc",
"smoothing_filter.h",
"sparse_fir_filter.cc",
@@ -62,6 +61,10 @@ rtc_static_library("common_audio") {
]
deps = [
+ ":sinc_resampler",
+ "..:webrtc_common",
+ "../base:gtest_prod",
+ "../base:rtc_base_approved",
"../system_wrappers",
]
public_deps = [
@@ -199,10 +202,33 @@ rtc_source_set("common_audio_c") {
}
public_configs = [ ":common_audio_config" ]
+ deps = [
+ "..:webrtc_common",
+ "../base:rtc_base_approved",
+ "../system_wrappers:system_wrappers",
+ ]
+}
+
+rtc_source_set("sinc_resampler") {
+ sources = [
+ "resampler/sinc_resampler.h",
+ ]
+ deps = [
+ "..:webrtc_common",
+ "../base:gtest_prod",
+ "../base:rtc_base_approved",
+ "../system_wrappers",
+ ]
}
if (current_cpu == "x86" || current_cpu == "x64") {
rtc_static_library("common_audio_sse2") {
+ # TODO(kjellander): Remove (bugs.webrtc.org/6828)
+ # Enabling GN check triggers dependency cycle:
+ # //webrtc/common_audio:common_audio ->
+ # //webrtc/common_audio:common_audio_sse2 ->
+ # //webrtc/common_audio:common_audio
+ check_includes = false
sources = [
"fir_filter_sse.cc",
"resampler/sinc_resampler_sse.cc",
@@ -216,11 +242,20 @@ if (current_cpu == "x86" || current_cpu == "x64") {
# Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
}
+ deps = [
+ ":sinc_resampler",
+ ]
}
}
if (rtc_build_with_neon) {
rtc_static_library("common_audio_neon") {
+ # TODO(kjellander): Remove (bugs.webrtc.org/6828)
+ # Enabling GN check triggers dependency cycle:
+ # //webrtc/common_audio:common_audio ->
+ # //webrtc/common_audio:common_audio_neon ->
+ # //webrtc/common_audio:common_audio
+ check_includes = false
sources = [
"fir_filter_neon.cc",
"resampler/sinc_resampler_neon.cc",
@@ -248,6 +283,10 @@ if (rtc_build_with_neon) {
suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
}
+ deps = [
+ ":sinc_resampler",
+ ]
+
public_deps = [
":common_audio_neon_c",
]
@@ -282,6 +321,10 @@ if (rtc_build_with_neon) {
# Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
}
+ deps = [
+ ":common_audio_c",
+ "../base:rtc_base_approved",
+ ]
}
}
@@ -335,6 +378,10 @@ if (rtc_include_tests) {
deps = [
":common_audio",
+ ":sinc_resampler",
+ "..:webrtc_common",
+ "../base:rtc_base_approved",
+ "../system_wrappers",
"../test:test_main",
"//testing/gmock",
"//testing/gtest",
« no previous file with comments | « .gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698