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
