[Scummvm-devel] Re: [Scummvm-cvs-logs] CVS: scummvm/sound midistreamer.cpp,1.1,1.2

[Max Horn]

At 1:09 Uhr -0800 18.11.2002, Jamieson Christian wrote:
>Update of /cvsroot/scummvm/scummvm/sound
>In directory sc8-pr-cvs1:/tmp/cvs-serv467/scummvm/sound
>
>Modified Files:
>	midistreamer.cpp
>Log Message:
>Added support for volume and pause control to Simon music.
>Note that MidiStreamer is now used as a streaming wrapper
>for ALL MidiDriver types, even those that natively support
>streaming. This is because MidiStreamer supports a hybrid
>of streamed and non-streamed MIDI, which is necessary to
>support interactive events.


If that is so, wouldn't it make more sense to just scrap the whole 
streaming API from the MidiDriver class, and instead of having 
MidiStreamer faking a "real" MidiDriver, just use it directly from 
the Simon API?

AFAICT the streaming stuff is only used for Simon anyway...


Max
-- 
-----------------------------------------------
Max Horn
Software Developer

email: <mailto:max at quendi.de>
phone: (+49) 6151-494890