Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#6244 closed defect (fixed)

TOLTECS: Invalid opcode 128!

Reported by: raziel- Owned by: johndoe123
Priority: high Component: Engine: Toltecs
Keywords: script Cc:
Game: 3 Skulls of the Toltecs

Description

ScummVM 1.6.0git (Feb 10 2013 09:07:47)
Features compiled in: Vorbis FLAC MP3 RGB zLib Theora AAC FreeType2

Using the map In the waterfall cave to whatever destination results in an
Invalid opcode 128!
and ScummVM quits.

3 Skulls of the Toltecs (DOS/English)

AmigaOS4 - PPC -SDL - BE
gcc (GCC) 4.2.4 (adtools build 20090118)

Ticket imported from: #3604025. Ticket imported from: bugs/6244.

Attachments (1)

toltecs.007 (34.7 KB) - added by raziel- 7 years ago.
Just use the map to go anywhere

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by raziel-

Attachment: toltecs.007 added

Just use the map to go anywhere

comment:1 Changed 6 years ago by sev-

This bug is nice to get fixed before the release. Raising priority for keeping the track.

comment:2 Changed 6 years ago by sev-

Priority: normalhigh

comment:3 Changed 6 years ago by sev-

Owner: set to johndoe123

comment:4 Changed 6 years ago by bluegr

This is a script bug, and it only manifests in that scene, AFAIK. So, you have to move one screen away to use the map, so it's not a game breaker, but it's annoying.

comment:5 Changed 6 years ago by johndoe123

After checking the original code it seems the original doesn't do anything when it encounters an invalid opcode. When the error handling here is disabled, i.e. an invalid opcode is simply ignored, the map works correctly. Since the rest of the game works regarding the opcodes it seems this is a bug in the original engine.
My suggestion is to ignore invalid opcodes like the original.

comment:6 Changed 6 years ago by bluegr

We are now ignoring invalid opcodes, like the original, as johndoe suggested. Fixed in commit 14f1529

comment:7 Changed 6 years ago by bluegr

Keywords: script added
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.