On Tue, Feb 23, 2016 at 8:22 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Feb 22, 2016 at 08:17:37PM -0500, Vittorio Giovara wrote: >> Sustain possible size changes via av_fast_malloc(). > > Sustain? That's not the right word, but I'm a tad unsure what you are > trying to say here.
If there is a parameter change we might need a bigger buffer, so I opted for this function that frees the old pointer and allocates a new one. >> --- a/libavcodec/intrax8.c >> +++ b/libavcodec/intrax8.c >> @@ -773,6 +773,7 @@ int ff_intrax8_decode_picture(IntraX8Context *const w, >> Picture *pict, >> { >> MpegEncContext *const s = w->s; >> int mb_xy; >> + int alloc_size = FFALIGN(FFABS(pict->f->linesize[0]) + 32, 32) * 2 * 24; > > That's a lot of magic numbers. indeed, they are the ones used in mpegvideo -- Vittorio _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel