RetroSearch Browse

Thu Jan 18 20:18:25 CET 2007 · https://ffmpeg.org/pipermail/ffmpeg-devel/2007-January/027614.html

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 rev).

Ramiro Polla

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://ffmpeg.org/pipermail/ffmpeg-devel/2007-January/027614.html below:

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