Opened 15 years ago

Closed 15 years ago

#1853 closed defect (invalid)

FOTAQ: 'WARNING: Trying to queue an invalid song number'

Reported by: SF/retroactivemind Owned by: joostp
Priority: normal Component: Engine: Queen
Keywords: Cc:
Game: Flight of the Amazon Queen

Description

Hi devs,

When using the latest CVS binary (0.7.0 Dec 20 2004 23:
56:57) with FOTAQ (English Full Talkie), i get the
following warning at the end of the car chase sequence:

"WARNING: Trying to queue an invalid song number 176,
max 106!"

With the CVS binary from Dec 16, the message read:

"WARNING: Expected MThd or GMD header but found ':ä♦
ü' instead!"

Ticket imported from: #1088437. Ticket imported from: bugs/1853.

Attachments (1)

queen.s00 (12.3 KB ) - added by SF/retroactivemind 15 years ago.
savegame - car chase sequence

Download all attachments as: .zip

Change History (13)

by SF/retroactivemind, 15 years ago

Attachment: queen.s00 added

savegame - car chase sequence

comment:1 by SF/retroactivemind, 15 years ago

Summary: WARNING: Trying to queue an invalid song number(FOTAQ) WARNING: Trying to queue an invalid song number

comment:2 by SF/retroactivemind, 15 years ago

Summary: (FOTAQ) WARNING: Trying to queue an invalid song numberFOTAQ: 'WARNING: Trying to queue an invalid song number'

comment:3 by cyxx, 15 years ago

I just added this warning. The culprit is this entry in the
Sound::_tune array :

/* 39 - Truck Fanfare */
{ { 67, 177, 0 }, { 0, 0 }, 2, 0 },

Song number 177 is obviously wrong, as the max number is 106
for the fotaq CD version. Maybe a song number > 128 has a
special meaning ? Any idea on this Joost ?

comment:4 by joostp, 15 years ago

IIRC only songs > 1000 are special commands.
All other should be interpreted as song numbers to be played.
I do remember something special about this fanfare, but
unfortunately my memory is failing me..
Quickly glancing at the original code raises more questions
than answers, so I'll have to think about this a bit.

comment:5 by SF/themmaster, 15 years ago

I've just checked against original interpreter and scummvm 8
times in a row, and there is no difference in original behaviour
and scummvm... exept that joe says "oil be back" during the
crash sound in scummvm and after it in original.

if someone proves me wrong, I'll eat my sock ;)

now, stop hunting ghosts and do something useful
joostp... :P

comment:6 by joostp, 15 years ago

I think what themmaster is trying to say is that the original interpreter
ignores this song and/or that there is no music missing.
So unless it is used to trigger a certain effect (that I'm unaware of). I
guess we should just keep our behaviour of ignoring it as well.

Will leave this open for a bit longer while I try to refresh my memory on
the subject and have confirmed this is indeed correct.

comment:7 by joostp, 15 years ago

Owner: set to joostp

comment:8 by cyxx, 15 years ago

Owner: joostp removed

comment:9 by cyxx, 15 years ago

Yes, it's quite possible the song number was just invalid,
that wouldn't surprise me much... The original interpreter
has quite a few issues like that (OOB and the likes...).

comment:10 by joostp, 15 years ago

Owner: set to joostp

comment:11 by joostp, 15 years ago

Resolution: invalid
Status: newclosed

comment:12 by joostp, 15 years ago

There doesn't seem to by anything in the original code that
deals with this, and as there is no difference in behaviour
(missing music or some effect), I'm convinced this is just a
bug in the original.
I've changed the warning() in CVS to a debug(3, ...); and am
closing this.

Note: See TracTickets for help on using tickets.