Opened 3 years ago
Closed 2 years 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 , 3 years ago
| Attachment: | Nancy Drew File 01 Secrets Can Kill (Windows English)_001.png added |
|---|
comment:1 by , 2 years ago
| Owner: | set to |
|---|---|
| Resolution: | → assigned |
| Status: | new → pending |
comment:2 by , 2 years 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 , 2 years ago
| Resolution: | assigned → fixed |
|---|---|
| Status: | pending → closed |
@fracturehill
Confirmed fixed :-D
Sorry for the delay
follow-up: 5 comment:4 by , 2 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → new |
Sorry, but the wrong colors are back :-/
Somehow a regression snuck in
comment:5 by , 2 years 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 , 2 years 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.