[FFmpeg-devel] [PATCH] Bink file demuxer and audio decoder

[pross at xvid.org]

On Fri, Jan 22, 2010 at 10:49:42PM +0100, Michael Niedermayer wrote:
> On Wed, Jan 20, 2010 at 09:15:24PM +1100, pross at xvid.org wrote:
> > On Thu, Jun 25, 2009 at 12:11:54AM +0200, Diego Biurrun wrote:
> > > On Sat, Jun 20, 2009 at 11:49:05PM +1000, Peter Ross wrote:

> > Fixes incorporated. Revised patchset enclosed.
> > 
> > -- Peter
> > (A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
> 
> >  smacker.c |   14 +++++++++++---
> >  1 file changed, 11 insertions(+), 3 deletions(-)
> > 35c4d928a5245a41d669ad2baa261044e7423496  smacker-decoder-r300.diff
> 
> > Index: libavformat/smacker.c
> 
> not maintained by me

Mike?

> 
> 
> > +    short *samples_end  = (void*)((uint8_t*)data + *data_size);
> 
> the void cast looks strange in there

Updated patch enclosed.  Thanks for spotting these.

Should the decoder be called binkaudio{1,2} or binkaudio{rdft,dct} ?
the 1 and 2 numbers were arbitrarily assigned by me.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: binkaudio-decoder-r5.diff
Type: text/x-diff
Size: 12020 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100127/3dd27b65/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100127/3dd27b65/attachment.pgp>