Opened 17 years ago

Closed 17 years ago

#631 closed defect (outdated)

ZAK256: Graphics bug in Mars Ankh room

Reported by: SF/dfabulich Owned by: fingolfin
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Zak McKracken

Description

When you enter the Ankh room on Mars (behind the middle
massive door in the Great Chamber), the palette looks
all messed up. In particular, your character looks
like he/she has turned to stone or something. (I've
included a screenshot/saved game for this bug.)

It does not impede play, but it looks quite bizarre.

I'm using the FM-TOWNS version with ScummVM 0.3.0b,
Built on Dec 6 2002 19:31:17 on Win2K.

Ticket imported from: #654284. Ticket imported from: bugs/631.

Attachments (2)

zakpal.png (77.1 KB ) - added by SF/dfabulich 17 years ago.
Screenshot of glitch
zak256.s12 (34.3 KB ) - added by SF/dfabulich 17 years ago.
Saved game in Ankh room with Melissa

Download all attachments as: .zip

Change History (10)

by SF/dfabulich, 17 years ago

Attachment: zakpal.png added

Screenshot of glitch

by SF/dfabulich, 17 years ago

Attachment: zak256.s12 added

Saved game in Ankh room with Melissa

comment:1 by SF/dfabulich, 17 years ago

+ saved game...

comment:2 by SF/ender, 17 years ago

This sounds like a regression caused by the reversion of
the copyPalColor() change back to a _currentPalette[a] = b op
(see script_v5.cpp revision 1.2)... although we are using the
shadowpalette now, so the effects are not quite as bad as
previously occured in bug 600287 (the reason for switching to
copyPalColor in the FIRST place).

Fingolfin, I remember you and Erik were talking about this one
recently, but I missed most of the conversation - why did you
revert this change? Do you think this is something that is simply
GF_OLD256 dependant, or more generic between all
SMALL_HEADER games, given the subopcode similarities?

comment:3 by SF/ender, 17 years ago

Owner: set to fingolfin

comment:4 by eriktorbjorn, 17 years ago

Using copyPalColors() caused regressions in MonkeyVGA (see
bug #647844, "MI1VGA: wrong color"), and probably worse ones
in LoomCD as well, although I never actually verified that
for myself.

I have no idea what the correct fix is, though.

comment:5 by fingolfin, 17 years ago

Owner: fingolfin removed

comment:6 by SF/dfabulich, 17 years ago

This bug appears to be closed in current snapshot: ScummVM
0.3.1cvs, Built on Dec 18 2002 01:12:15. Now works properly.

comment:7 by fingolfin, 17 years ago

So, can this be closed then?

comment:8 by fingolfin, 17 years ago

Owner: set to fingolfin
Resolution: outdated
Status: newclosed
Note: See TracTickets for help on using tickets.