[Ffmpeg-devel] [PATCH] return correct decoded size in amr.c

Benoit Fouet benoit.fouet
Wed Apr 4 08:39:58 CEST 2007


Hi,

Michael Niedermayer wrote:
> Hi
>
> On Tue, Apr 03, 2007 at 09:43:29PM +0200, Benoit Fouet wrote:
>   
>> Hi,
>>
>> following this thread (actually, a part of it, as the OP broke it :) ):
>> http://lists.mplayerhq.hu/pipermail/ffmpeg-user/2007-April/007859.html
>> here is a patch to amr.c against r8618
>>
>> i hesitated between changing the:
>> "*data_size+=160*2;" to "*data_size=160*2"
>> or initialize "*data_size=0" at the beginning of the function
>>
>> i chose the latter, as for what i understand, decoder is called with
>> only one frame at a time...
>>     
>
> looks ok assuming it works
>
>   
i've tested it on all bitrates here, didn't see any problem, if someone
want to give it a try to confirm, i can provide samples.

Ben




More information about the ffmpeg-devel mailing list