[Scummvm-devel] IMA codec in iMuseDigital (Was: CVS mumble jumble)

[Max Horn]

Am 10.11.2004 um 07:57 schrieb Pawel Kolodziejski:

>> Update of /cvsroot/scummvm/scummvm/scumm/imuse_digi
>> In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv4785
>>
>> Modified Files:
>> 	dimuse_codecs.cpp
>> Log Message:
>
> of course it is, also other variant of IMA is in residual.
>
Ah fine, but then why is it not already documented?

Note that we spent countless hours to manually translate every bit of 
this code from assembly. I spent a lot of time fixing memory overwrites 
in this code, too... if I had known the IMA bit, it would have helped a 
lot ...

Also, it appears as if the code could probably be simplified. Looking 
at it, those lookup tables may not actually be necessary on modern 
machines (on todays machines, doing a few extra muls/shifts definitely 
beats using a memory lookup table and branches...)


Cheers,

Max