Opened 16 years ago

Closed 16 years ago

#1260 closed defect (fixed)

INDY3TOWNS: Pallette issues in Brunwald Castle?

Reported by: SF/cloney Owned by: Kirben
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Indiana Jones 3

Description

ScummVM 0.5.3cvs (Oct 3 2003 11:39:37), Win32.

I've not actually played this on an FM Towns, so I'm
going by comparison to the PC VGA version.

In Brunwald Castle, the guards on the overhead-shot
seem to be wearing grey or off-white uniforms, rather
than green. When you meet them to talk/fight, they're
in green, so I figure this must be a pallette issue
rather than a design choice. A png of this is attached.

Also, in the room with the suit of armour holding an
axe, the area on the floor where the axe hits is bright
pink, even before you've pushed the axe. Once you do,
it leaves a dull grey cut, as in the PC version.

Cloney

Ticket imported from: #817267. Ticket imported from: bugs/1260.

Attachments (2)

indyshot.png (7.5 KB ) - added by SF/cloney 16 years ago.
Screen-grab
indy3towns.s03 (58.0 KB ) - added by SF/cloney 16 years ago.
Save game in Brunwald

Download all attachments as: .zip

Change History (6)

by SF/cloney, 16 years ago

Attachment: indyshot.png added

Screen-grab

by SF/cloney, 16 years ago

Attachment: indy3towns.s03 added

Save game in Brunwald

comment:1 by SF/hibernatus, 16 years ago

The grey uniforms are normal.
On the other hand the text color when they talk is wrong, but
there's another bug report about it (16 constant colors).
About the room with the axe, it's a problem of roomOps case
2 which shouldn't do the same thing as case 4.
On FM Towns the 16 first colors of the palette are constant,
and roomOps case 2 seems to remap them when they're used
while case 4 seems to remap them in the palette. So i
suppose you'd need to ignore case 4 when the color index is
below 16.

What happens in Indy3 is that the entry script does either
RoomColor(4,13) or RoomColor(8,13), but another script
seems to reset the first colors (_shadowColor[i] = i) but with
roomOps case 4 instead of case 2.

comment:2 by SF/hibernatus, 16 years ago

Hmm, i'm really sorry, i didn't understand that whole thing :)
case 4 does something under unz, as well as case 2, but
anyway it's not the same function and case 2 should have its
own _shadowPalette (i saw there's something called
_roomPalette for 16 colors games, might be the same thing?).

comment:3 by SF/cloney, 16 years ago

Looks like the changes Kirben made last night have fixed the
bright pink floor problem.

Why did they make the uniforms grey? It seems very weird.

Cloney

comment:4 by Kirben, 16 years ago

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