Opened 18 years ago

Closed 18 years ago

Last modified 2 years ago

#8228 closed patch

SIMON2: Is this what realizePalette_unk() should do?

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

Description

From looking at the disabled realizePalette_unk() function, it looks to me as if it's simply trying to fade in the picture, albeit in a slightly peculiar way.

At the very least, it's missing a call to _system->update_screen(), and I don't understand why it compares *src to i and not to *dst.

This patch is what I came up with while experimenting with it, but I've never seen what the original looks like, and it could probably use some further tweaking anyway.

Ticket imported from: #740321. Ticket imported from: patches/333.

Attachments (1)

simon-unk-palette.diff (3.2 KB ) - added by eriktorbjorn 18 years ago.
Patch against a May 19 CVS snapshot

Download all attachments as: .zip

Change History (4)

by eriktorbjorn, 18 years ago

Attachment: simon-unk-palette.diff added

Patch against a May 19 CVS snapshot

comment:1 by Kirben, 18 years ago

Owner: set to Kirben
Status: newclosed

comment:2 by Kirben, 18 years ago

The patch works fine, although there is slight pause in Lion/Simon after each fade in. Patch has been commited to ScummVM cvs. I can't tell if original simon 2 actually had a fade in/out or not, as it always runs too fast.

comment:3 by digitall, 2 years ago

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