[Ffmpeg-devel] [PATCH] a FIXME in lavc/utils.c

angustia at arrozcru.no-ip.org angustia
Thu Jan 18 20:17:31 CET 2007


Hello,

Quoting Michael Niedermayer <michaelni at gmx.at>:

> Hi
>
> On Thu, Jan 18, 2007 at 03:49:38AM -0200, ramiro at lisha.ufsc.br wrote:
>> Hello,
>>
>> Is this the correct solution for this FIXME?
>> Regression tests pass.
>>
>> Ramiro Polla
>> Index: libavcodec/utils.c
>> ===================================================================
>> --- libavcodec/utils.c	(revision 7576)
>> +++ libavcodec/utils.c	(working copy)
>> @@ -301,7 +301,7 @@
>>
>>              buf->linesize[i]= picture.linesize[i];
>>
>> -            buf->base[i]= av_malloc(size[i]+16); //FIXME 16
>> +            buf->base[i]= av_malloc(size[i]+FF_INPUT_BUFFER_PADDING_SIZE);
>
> NO!
>
> 16 is the macroblock size here
>

Ok... I guess I shouldn't play around with stuff I still don't know =)
Anyways, that FIXME has been there since rev 1300 something (forgot  
the exact number).

Ramiro Polla




More information about the ffmpeg-devel mailing list