[Scummvm-devel] Re: [Scummvm-cvs-logs] CVS: scummvm/sky adlibchannel.h,1.1,1.2 adlibchannel.cpp,1.1,1.2 adlibmusic.cpp,1.1,1.2 adlibmusic.h,1.1,1.2

[Jonathan Gray]

On Sat, 10 May 2003, Arisme wrote:

> Ok, back to the new adlib driver topic after some more tests ... I'm now
> convinced that it'll never run smoothly (at least not as the previous one)
> on slow devices (WinCE and possibly others)  - the new approach taken to
> emulate the clock is too CPU consuming (see "advancex"), without considering
> optimizations on the sampling quality  ; on a SPV (120 MHz), the games are
> barely playable and way out of synch (even the "non IMuse" games as this
> code is called from the mixer :)), on an Ipaq (200 MHz), it's almost OK for
> IMuse games, some sequences are still out of synch, but it's too slow for
> Midi games.
>
> What do you want to do for the upcoming release ?
>
> 1) We revert to the old adlib code (/me happy but not the other contributors
> I guess :))

This has already been done on the 0.4.0 branch, however the long term
choice has yet to be decided I think...

>
> 2) We keep the new code (reverse previous)
>
> 3) We keep both codes and link to the old one for slow devices (assuming the
> old code wasn't breaking anything in IMuse, it looked OK given all the tests
> done in the branch)

I'd probably be leaning towards this, as the newer version does sound a
bit more accurate in a few places.

>
> 4) ...
>

Jonathan