[pulseaudio-discuss] [PATCH] echo-cancel: Fix assert with webrtc's built-in drift compensation
Tanu Kaskinen
tanuk at iki.fi
Fri Mar 17 20:11:58 UTC 2017
On Fri, 2017-03-17 at 07:59 +0530, Arun Raghavan wrote:
> The webrtc canceller seems to have changed to require that the
> set_stream_drift_samples() method be called before every call of
> ProcessStream().
>
> So we now call ec->set_stream_drift_samples() before calling
> ec->record() by:
>
> 1. Always calling do_push_drift_comp() instead of only when the sink is
> running
>
> 2. Calling set_stream_drift_samples() in the loop with record() instead
> of outside
>
> We do kind of leak this quirk of the webrtc canceller into the generic
> bits of module-echo-cancel, but this should not be harmful in the
> general case either.
> ---
> src/modules/echo-cancel/module-echo-cancel.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
Looks good to me.
--
Tanu
https://www.patreon.com/tanuk
More information about the pulseaudio-discuss
mailing list