Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#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 RayKoopa)

  • 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)

image-20220522-105750.png (30.3 KB ) - added by RayKoopa 2 years ago.

Download all attachments as: .zip

Change History (9)

by RayKoopa, 2 years ago

Attachment: image-20220522-105750.png added

comment:1 by RayKoopa, 2 years ago

Description: modified (diff)

comment:2 by RayKoopa, 2 years ago

Cc: RayKoopa added

comment:3 by sluicebox, 2 years ago

Priority: lownormal

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 =)

comment:4 by RayKoopa, 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 sluicebox, 2 years ago

Owner: set to sluicebox
Resolution: fixed
Status: newclosed

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 RayKoopa, 2 years ago

You deserve the accomplishment. It seems to have to do with unitialized memory?

Version 0, edited 2 years ago by RayKoopa (next)

comment:7 by sluicebox, 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 RayKoopa, 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.

Note: See TracTickets for help on using tickets.