#13497 closed defect (fixed)
LSL6 inventory items have incorrectly colored background
Reported by: | RayKoopa | Owned by: | sluicebox |
---|---|---|---|
Priority: | normal | Component: | Engine: SCI |
Version: | Keywords: | ||
Cc: | RayKoopa | Game: | Leisure Suit Larry 6 |
Description (last modified by )
- ScummVM 2.6.0git5915-gf29cdfcf46a (May 22 2022 09:53:09)
- Game language: English
- Game version: Hi-Res 1.000.000
- Windows 11 21H2 10.0.22000.675
Steps to reproduce:
- Start a new game.
- Look at the inventory panel at the top right.
- Optionally grab an item and notice it still has a black background.
The background color behind item graphics in the inventory panel at the top right is pure black (hex 000000) while it should match the surrounding panel's dark green color (hex 003B3B).
Attachments (1)
Change History (9)
by , 2 years ago
Attachment: | image-20220522-105750.png added |
---|
comment:1 by , 2 years ago
Description: | modified (diff) |
---|
comment:2 by , 2 years ago
Cc: | added |
---|
comment:3 by , 2 years ago
Priority: | low → normal |
---|
comment:4 by , 2 years ago
Sure, go ahead :) I didn't want to bug people too much about a relatively small thing like this so I created it with "Low".
comment:5 by , 2 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Fixed in: https://github.com/scummvm/scummvm/commit/b74b93ad46e0d3e0c88f632b90f1c04bf81512a5
Well it turned out to be easy but I'm still going to cling to that sense of accomplishment =)
Thanks again for reporting. Good eye!
comment:6 by , 2 years ago
You deserve the accomplishment. It seems to have to do with unitialized memory?
comment:7 by , 2 years ago
Yes; it works by accident in the original.
The script chooses green or black based on a variable that is never set, so it's operating on "whatever happens to be there". We detect these errors and have a table of known offenders so we can reply with the value they need to work. Our workaround for this one was wrong, we were offering a zero but really it needs a non-zero value for the correct outcome.
comment:8 by , 2 years ago
That's awesome! I enjoy how high level that fix for such a low level thing is. Thanks again for checking and fixing it.
Hello! Thank you for reporting this. I confirmed with a playthrough video that indeed the inventory background is teal and not black: https://www.youtube.com/watch?v=0Bj3TSdYZNY
Mac version is also drawing black in ScummVM.
Bumping this to normal so that I can feel a greater sense of accomplishment if I fix it =)