Opened 22 years ago

Closed 22 years ago

#188 closed defect (fixed)

indy 4 crash at start

Reported by: SF/logicdeluxe Owned by: SF/ender
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Indiana Jones 4

Description

Built on Apr 25 2002 21:10:18 win talkie version

The Indy logo is messed up and the moment Indy should appear Scummvm terminates: Detected game 'Indiana Jones 4 and the Fate of Atlantis', version 5.5.0 WARNING: roomops:14 load-string(31,"iq-points") not implemented! Loading room 68 Adding object 933 from room 68 into inventory Adding object 937 from room 68 into inventory Adding object 938 from room 68 into inventory Loading room 0 Loading room 4 Error(4:206:0xD877): Value 1024 is out of bounds (0,799) int script(206) msg Variable %d out of range(r)!

It worked with Built on Apr 23 2002 07:37:56

Ticket imported from: #548672. Ticket imported from: bugs/188.

Change History (4)

comment:1 by SF/mnepomuk, 22 years ago

Found. Reason is the ugly handling of '_gameId' in if- statements, where a logical AND is used (&) instead of a direct comparance (==). So the result of the if-statements are a matter of choosing the "right" values in the enum- gameId statement, which is crap. There are several occurance of that spreaded over some files.

Either have to switch to a binary driven gameId (I guess this is not necessary) or correct the &'s to =='s. I cannot decide this.

Mikesch

P.S.: Shame: I made the same mistakes in my patches ;-(

comment:2 by SF/mnepomuk, 22 years ago

Found. Reason is the ugly handling of '_gameId' in if- statements, where a logical AND is used (&) instead of a direct comparance (==). So the result of the if-statements are a matter of choosing the "right" values in the enum- gameId statement, which is crap. There are several occurance of that spreaded over some files.

Either have to switch to a binary driven gameId (I guess this is not necessary) or correct the &'s to =='s. I cannot decide this.

Mikesch

P.S.: Shame: I made the same mistakes in my patches ;-(

comment:3 by SF/ender, 22 years ago

Owner: set to SF/ender
Resolution: fixed
Status: newclosed

comment:4 by SF/ender, 22 years ago

Fixed.

Note: See TracTickets for help on using tickets.