Am So., 5. Apr. 2020 um 17:14 Uhr schrieb James Almer <jamr...@gmail.com>:
>
> On 4/5/2020 12:06 PM, Carl Eugen Hoyos wrote:
> > Hi!
> >
> > Attached patch simplifies a funtion in the amrwb decoder, as suggested by
> > James.
> >
> > Please comment, Carl Eugen
>
> > From 91d50f466728d7d0680882f6f63217f39808c2af Mon Sep 17 00:00:00 2001
> > From: Carl Eugen Hoyos <ceffm...@gmail.com>
> > Date: Sun, 5 Apr 2020 17:04:44 +0200
> > Subject: [PATCH] lavc/amrwbdec: Use av_samples_set_silence().
> >
> > Suggested-by: James Almer
> > ---
> > libavcodec/amrwbdec.c | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/libavcodec/amrwbdec.c b/libavcodec/amrwbdec.c
> > index b488a5d3c7..1d4f00e717 100644
> > --- a/libavcodec/amrwbdec.c
> > +++ b/libavcodec/amrwbdec.c
> > @@ -1122,8 +1122,7 @@ static int amrwb_decode_frame(AVCodecContext *avctx,
> > void *data,
> > expected_fr_size = ((cf_sizes_wb[ctx->fr_cur_mode] + 7) >> 3) + 1;
> >
> > if (ctx->fr_cur_mode == NO_DATA) {
> > - for (i = 0; i < frame->nb_samples; i++)
> > - buf_out[i] = 0.f;
> > + av_samples_set_silence(&buf_out, 0, frame->nb_samples, 1,
> > AV_SAMPLE_FMT_FLT);
>
> Should be frame->data (or frame->extended_data), not &buf_out. The
> function expects an uint8_t**, not float**.
>
> LGTM otherwise.
Applied with that change.
Thank you, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
