From c5e90d7d51e70b95527ea4e6293df4800a34db67 Mon Sep 17 00:00:00 2001 From: Arun Raghavan Date: Wed, 9 Jul 2025 12:48:24 -0400 Subject: [PATCH] spa: aec: webrtc: Drop outdated comment The Intelligibility enhancer was removed, so the FIXMEs are irrelevant. --- spa/plugins/aec/aec-webrtc.cpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/spa/plugins/aec/aec-webrtc.cpp b/spa/plugins/aec/aec-webrtc.cpp index e20991d5b..6ad3a92e0 100644 --- a/spa/plugins/aec/aec-webrtc.cpp +++ b/spa/plugins/aec/aec-webrtc.cpp @@ -124,10 +124,6 @@ static int webrtc_init2(void *object, const struct spa_dict *args, // result in very poor performance, disable by default bool gain_control = webrtc_get_spa_bool(args, "webrtc.gain_control", false); - // FIXME: Intelligibility enhancer is not currently supported - // This filter will modify playback buffer (when calling ProcessReverseStream), but now - // playback buffer modifications are discarded. - #if defined(HAVE_WEBRTC) webrtc::Config config; config.Set(new webrtc::ExtendedFilter(extended_filter)); @@ -309,9 +305,6 @@ static int webrtc_run(void *object, const float *rec[], const float *play[], flo for (size_t j = 0; j < impl->out_info.channels; j++) impl->out_buffer[j] = out[j] + out_config.num_frames() * i; - /* FIXME: ProcessReverseStream may change the playback buffer, in which - * case we should use that, if we ever expose the intelligibility - * enhancer */ if ((res = impl->apm->ProcessReverseStream(impl->play_buffer.get(), play_config, play_config, impl->play_buffer.get())) != webrtc::AudioProcessing::kNoError) {