On Mon, 2016-01-25 at 09:36 +0530, Arun Raghavan wrote: > On 24 January 2016 at 22:00, Tanu Kaskinen <[email protected]> wrote: > > On Mon, 2016-01-18 at 13:06 +0530, [email protected] wrote: > > > @@ -253,7 +263,7 @@ void pa_webrtc_ec_play(pa_echo_canceller *ec, const > > > uint8_t *play) { > > > pa_assert(play_frame.samples_per_channel_ <= > > > webrtc::AudioFrame::kMaxDataSizeSamples); > > > memcpy(play_frame.data_, play, ec->params.priv.webrtc.blocksize); > > > > > > - apm->AnalyzeReverseStream(&play_frame); > > > + apm->ProcessReverseStream(&play_frame); > > > > This looks like a potentially unrelated change. Why is this change > > done? > > It is needed for the intelligibility enhancer to work, but I later > realised we don't actually support this -- unlike > AnalyzeReverseStream(), ProcessReverseStream() modifies the playback > samples. This is something that needs to be done in the future > (there's a later commit that disables this that I could not squash due > to some intermediate changes).
Does that mean that the intelligibility enhancer feature doesn't work at all? In that case this patch should be dropped. -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
