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

[Michael Niedermayer]

Hi

On Thu, Apr 05, 2007 at 09:05:27AM +0200, Benoit Fouet wrote:
> 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 :) )

patch still looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070405/e233238a/attachment.pgp>