Opened 4 years ago

Closed 2 years ago

#10734 closed defect (fixed)

SCI: LB1: Turn-crank and open-vault sound fx not playing

Reported by: sluicebox Owned by: sluicebox
Priority: normal Component: Engine: SCI
Version: Keywords:
Cc: Game: Laura Bow 1: The Colonel's Bequest

Description

ScummVM isn't playing the sound effects in LB1 when turning the crank in the basement to open the secret passage or when opening/closing a vault in the crypt.

Both actions log:
WARNING: MIDI parser: the requested SCI0 sound has no MIDI note data for the currently selected sound driver!

Occurs on DOS and Atari ST versions. Amiga works.

Using default audio settings in ScummVM. Win7x64, multiple machines, verified resource files are correct from floppies and collection CDs. Occurs in nightly build, 2.0, 1.9, and 1.8. (That's as far back as I checked). These sound effects play on Sierra's interpreter.

Attached are DOS saves at the crank and vault. "turn crank", "open vault with crowbar".

Attachments (2)

laurabow.009 (30.6 KB ) - added by sluicebox 4 years ago.
laurabow.010 (30.1 KB ) - added by sluicebox 4 years ago.

Download all attachments as: .zip

Change History (4)

by sluicebox, 4 years ago

Attachment: laurabow.009 added

by sluicebox, 4 years ago

Attachment: laurabow.010 added

comment:1 by sluicebox, 2 years ago

I've narrowed this down some.

Sound 123 (pry vault) and sound 126 (turn crank) have two channels:

channel		flags
00		30
09		0F

Neither of these channels has the Adlib flag 04 set and so neither are played by ScummVM when using Adlib, but the original Adlib driver does play this channel. At least, it does for these sounds.

To test this, I changed MidiPlayer_Adlib::hasRhythmChannel() to return true, causing channel 9 to be played. Both sounds played correctly as they do in the original.

So now the question is why does Sierra's Adlib driver play this?

Version 1, edited 2 years ago by sluicebox (previous) (next) (diff)

comment:2 by sluicebox, 2 years ago

Owner: set to sluicebox
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.