Opened 21 years ago

Closed 21 years ago

#1260 closed defect (fixed)

INDY3TOWNS: Pallette issues in Brunwald Castle?

Reported by: SF/cloney Owned by: Kirben
Priority: normal Component: Engine: SCUMM
Version: 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 21 years ago.
Screen-grab
indy3towns.s03 (58.0 KB ) - added by SF/cloney 21 years ago.
Save game in Brunwald

Download all attachments as: .zip

Change History (6)

by SF/cloney, 21 years ago

Attachment: indyshot.png added

Screen-grab

by SF/cloney, 21 years ago

Attachment: indy3towns.s03 added

Save game in Brunwald

comment:1 by SF/hibernatus, 21 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, 21 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, 21 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, 21 years ago

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