[Scummvm-tracker] [ScummVM :: Bugs] #15269: Leisure Suit Larry 5 copy protection screen broken in ScummVM 2.8.1
ScummVM :: Bugs
trac at scummvm.org
Mon Jul 15 05:57:07 UTC 2024
#15269: Leisure Suit Larry 5 copy protection screen broken in ScummVM 2.8.1
-------------------------+-----------------------------------
Reporter: miracleflame | Owner: sluicebox
Type: defect | Status: closed
Priority: normal | Component: Engine: SCI
Version: | Resolution: invalid
Keywords: | Game: Leisure Suit Larry 5
-------------------------+-----------------------------------
Changes (by sluicebox):
* owner: (none) => sluicebox
* resolution: => invalid
* status: new => closed
Comment:
This is a SCI Companion compiler bug, not a ScummVM bug.
This error does not occur when running the real game, it occurred when
running a recompiled script. If a ScummVM save were attached, we would
have seen this because we wouldn't have been able to load it. You were
probably just confused over which game directory you were pointed at; that
happens to me all the time. Otherwise, "error occurs in a mod I'm making"
is the kind of info you want to include in a ticket =)
SCI Companion's decompiler produced a script that its compiler doesn't
accurately compile. That's script 20. The decompiler produced the `btn6`
object definition without a `keyStr` property definition. That part is
correct; property values are inherited from their superclass by default,
and in this case both share the same string "x". But Companion's compiler
does not handle that correctly; it omits the relocation table entry for
`btn6:keyStr`. The relocation is what makes the difference between an
integer and a pointer (reference). Without it, the offset to the string
"x" is treated as just an integer. That's exactly what the ScummVM error
reports, that the first kFormat parameter is an integer instead of a
reference.
--
Ticket URL: <https://bugs.scummvm.org/ticket/15269#comment:4>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM
More information about the Scummvm-tracker
mailing list