Opened 18 months ago
Closed 13 months ago
#14472 closed defect (fixed)
NANCY: SCK - Wrong colors on BE target
Reported by: | raziel- | Owned by: | fracturehill |
---|---|---|---|
Priority: | normal | Component: | Engine: Nancy |
Version: | Keywords: | endian conversion | |
Cc: | Game: | Nancy Drew: Secrets Can Kill |
Description
ScummVM 2.8.0git (May 15 2023 22:17:09)
Using SDL backend with SDL 2.26.5
Features compiled in: Vorbis FLAC MP3 RGB zLib MPEG2 MikMod Theora VPX AAC A/52 FreeType2 FriBiDi JPEG PNG GIF cloud (servers, local) ENet SDL2 TinyGL OpenGL (with shaders)
See attached screenshot, color conversion gone wrong (if there is any)
Nancy Drew: Secrets Can Kill (Windows/English)
AmigaOS4 - PPC - Big Endian
gcc (adtools build 11.3.0) 11.3.0
Attachments (1)
Change History (8)
by , 18 months ago
Attachment: | Nancy Drew File 01 Secrets Can Kill (Windows English)_001.png added |
---|
comment:1 by , 14 months ago
Owner: | set to |
---|---|
Resolution: | → assigned |
Status: | new → pending |
comment:2 by , 14 months ago
No problem, thank you very much.
I will test asap...unfortunately that will have to wait till october since im back at work, sorry :-/
comment:3 by , 14 months ago
Resolution: | assigned → fixed |
---|---|
Status: | pending → closed |
@fracturehill
Confirmed fixed :-D
Sorry for the delay
follow-up: 5 comment:4 by , 13 months ago
Resolution: | fixed |
---|---|
Status: | closed → new |
Sorry, but the wrong colors are back :-/
Somehow a regression snuck in
comment:5 by , 13 months ago
Yes, I rewrote the engine's resource management last week and apparently broke the LE->BE conversion. I've pushed a fix to master, let me know if everything works correctly.
comment:7 by , 13 months ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
No, thank you for testing! I'm re-marking this as resolved, hopefully this will be the last time I touch this ticket :)
I'm obscenely late on responding to this, my apologies :)
The issue seems to be a simple case of image/video data endianness not being converted. I've just pushed what I believe is a fix, but I don't have a big endian machine to test it on, so I'd appreciate if you gave it a try to confirm.