[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