417
edits
(Changed wording.) |
(Added info on done work during the night between 2007-08-19 and 2007-08-20.) |
||
Line 1,304: | Line 1,304: | ||
of sequence (0xFC) before reading the delta-time as some files do use 0xFC | of sequence (0xFC) before reading the delta-time as some files do use 0xFC | ||
where the delta-time would be. | where the delta-time would be. | ||
=== I'm going bughunting... be vevy vevy quiet! === | |||
I wasn't planning on going bughunting but as I tried to submit | |||
my MIDI parsing/stepping-through code into SVN I found out that | |||
sound didn't work with AGI in the SVN. Possibly Fingolfin inadvertently | |||
broke it or made the bug somehow come up more easily (At least AFAIK it | |||
didn't crash like that before ;-)). Well, in the wee hours I disgruntently | |||
tried to fix the bug and at last I succeeded (Here's the [http://scummvm.svn.sourceforge.net/viewvc/scummvm/scummvm/trunk/engines/agi/sound.cpp?r1=28671&r2=28672&sortby=date diff] | |||
for the fix. The crash was happening because the PCjr sound mixing code tried to use uninitialized values (e.g. phase, ins) when mixSound was called before anything was playing). | |||
After fixing the crashing bug I got to the point where I could commit | |||
the MIDI parsing/stepping-through code into the SVN. Whee, it's not | |||
perfect (Speed's probably off), it doesn't use the MIDI classes yet etc, | |||
but it steps through the MIDI files (That aren't totally standard) hopefully | |||
quite correctly :). |
edits