Opened 4 years ago
Closed 4 years ago
#11476 closed defect (fixed)
QFG1: Incomplete Music
Reported by: | Deledrius | Owned by: | athrxx |
---|---|---|---|
Priority: | normal | Component: | Audio: MT32 |
Version: | Keywords: | mt-32, qfg1, qfg1ega | |
Cc: | Deledrius | Game: | Quest for Glory 1 |
Description
Some music queues appear to be missing when playing QfG1 (EGA) in Emulated MT-32 mode on the current version of ScummVM. I noticed they played correctly on an older daily build (2.1.0git4992-g1b333ea19a March 25 2019) but do not work on the 2.1.0 release or newer, so this would seem to be a regression.
This can be easily tested right from the start of the game by entering Zara's Magic Shop: the music should start quite loudly and jarringly. Instead, only some very low quiet instruments play after a long delay. Another instance available at the start of the game is Brauggi's music, which only plays the beginning but not the coda.
Running git bisect, I found that 037eec31446b64ec56b24e31d5be392390a694ba is the first bad commit.
To ease testing, I have attached a saved game immediately outside the Magic Shop. Just enter and listen for music (or silence).
Attachments (1)
Change History (8)
by , 4 years ago
comment:1 by , 4 years ago
I'm not sure if this is the result of the same bug, but I also noticed that the music in Adlib mode sounds wrong here as well. In the current release, the music seems to only be playing a single note at a time, whereas it plays chords in the 2.1.0git4992-g1b333ea19a build. Alas, I did not test this mode when performing the git bisect, only the MT-32 mode.
comment:2 by , 4 years ago
Thanks for your report. Is the issue limited to QFG? In that case I'd focus on that...
comment:3 by , 4 years ago
Adlib issue solved now in https://github.com/scummvm/scummvm/pull/2262
EDIT
====
Too early, it's not fixed yet...
comment:4 by , 4 years ago
This magic shop is an interesting test case. That sound would fail on most devices I tested (MT-32, AdLib, CMS, FB-01). It is the same underlying issue. I think I've tracked it down. My local attempt of a fix seems to work.
Just to make sure we're talking about the same issue. Could you please try to comment out line 581 in music.cpp
if (pSnd->status != kSoundPaused)
and tell me if that does it for you (this is not the actual complete fix, but should address the issue).
comment:5 by , 4 years ago
Yes, both Zara's shop and Brauggi's song play correctly with that workaround in place.
comment:6 by , 4 years ago
Thanks for testing! I don't even know what Brauggi's song is, since I've never played the game ;-).
I'll do some checkups with some other interpreter versions and commit a fix shortly.
comment:7 by , 4 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I close this, since the fix has been merged and you have confirmed it to be working on GitHub.
QfG1 EGA Saved Game - Outside Magic Shop