Opened 22 years ago

Closed 22 years ago

Last modified 5 years ago

#8012 closed patch

SIMON2: More lockup workarounds

Reported by: eriktorbjorn Owned by: SF/ender
Priority: normal Component: Engine: AGOS
Version: Keywords:
Cc: Game: Simon the Sorcerer 2

Description

Once again I was bitten by a Simon function grabbing the lock and then returning without releasing it.

This time it happened in the save game dialog (which doesn't seem to work too well at the moment anyway, but that's another issue), where ScummVM printed an "Invalid coordinates in video_toggle_colors" message. This probably means there's another bug in there somewhere, but it still shouldn't lock the interpreter.

The patch should fix a few other similar errors, but I've never actually triggered any of those so that part of the patch is guesswork.

Ticket imported from: #595414. Ticket imported from: patches/117.

Attachments (1)

simonlock.diff (1.0 KB ) - added by eriktorbjorn 22 years ago.
Patch against August 14 CVS snapshot

Download all attachments as: .zip

Change History (3)

by eriktorbjorn, 22 years ago

Attachment: simonlock.diff added

Patch against August 14 CVS snapshot

comment:1 by SF/ender, 22 years ago

Owner: set to SF/ender
Status: newclosed

comment:2 by digitall, 5 years ago

Component: Engine: AGOS
Game: Simon the Sorcerer 2
Note: See TracTickets for help on using tickets.