[pulseaudio-discuss] [PATCH v4 17/23] echo-cancel: Improve webrtc canceller error handling a bit

Arun Raghavan arun at accosted.net
Mon Feb 22 06:25:03 UTC 2016


On Sat, 2016-02-20 at 14:53 +0200, Tanu Kaskinen wrote:
> On Wed, 2016-02-17 at 19:47 +0530, arun at accosted.net wrote:
> > @@ -363,7 +367,8 @@ 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.webrtc.blocksize * pa_frame_size(ss));
> >  
> > -    apm->ProcessReverseStream(&play_frame);
> > +    if (apm->ProcessReverseStream(&play_frame) != webrtc::AudioProcessing::kNoError)
> > +        pa_log("Failed to process playback stream");
> 
> This should have been changed to an assertion too. I have it changed in
> my local tree.

Erf, missed that. Thanks for catching it.

-- Arun


More information about the pulseaudio-discuss mailing list