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

[Benoit Fouet]

Hi,

Benoit Fouet wrote:
> 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.
>   
well, here is the updated patch against svn head (as the other patches
have been applied before this one :) )

Ben

-------------- next part --------------
A non-text attachment was scrubbed...
Name: amr.c.patch
Type: text/x-patch
Size: 399 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070405/5ef5b63c/attachment.bin>