[FFmpeg-devel] [PATCH] scale msadpcm table to fit in int8_t

Michael Niedermayer michaelni
Sat Jul 12 03:48:38 CEST 2008

On Fri, Jul 11, 2008 at 06:14:44PM -0700, Mike Melanson wrote:
> Reimar D?ffinger wrote:
> > Hello,
> > this implements the same change I did to the MPlayer decoder.
> > Saves only a few bytes though and would probably be slower on an 8-bit
> > CPU without good shift support (e.g. Atmel AVR) if someone cares...
> I guess ADPCM algorithms are not specified to be bit exact? This changes 
> the output from before. But as long as it sounds right, I can update the 
> relevant FATE tests. However, you will be responsible for updating the 
> regression tests (which are presently broken due to this change).

i thought it was a bitexact optim just reducing the table sizes
has reimar maybe missed some use of the tables? :)

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- 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/20080712/4db63607/attachment.pgp>

More information about the ffmpeg-devel mailing list