[Scummvm-tracker] [ScummVM :: Bugs] #832: FOA: Invalid actor XXX in o5_getActorRoom() (was: FOA: Invalid actor)

[ScummVM :: Bugs]

#832: FOA: Invalid actor XXX in o5_getActorRoom()
---------------------------+------------------------------
Reporter:  SF/grandepuffo  |       Owner:  fingolfin
    Type:  defect          |      Status:  new
Priority:  normal          |   Component:  Engine: SCUMM
 Version:                  |  Resolution:
Keywords:  script          |        Game:  Indiana Jones 4
---------------------------+------------------------------
Changes (by dwatteau):

 * status:  closed => new
 * resolution:  fixed =>
 * summary:  FOA: Invalid actor => FOA: Invalid actor XXX in
              o5_getActorRoom()

Comment:

 Reopening this very old ticket, as suggested by AndywinXp ("This way I
 will eventually remember to check for the correct behavior in my v5 disasm
 and properly fix it") after this commit of mine:
 https://github.com/scummvm/scummvm/commit/9c78d324a6a674d022d725cc05aee57de35fd216

 It appears that we still need this workaround as of 2022. A quick test for
 this is the `5234` boot param.

 If I test the original interpreter with DREAMM, I don't see any crash,
 while ScummVM will issue a fatal error about this. I can't say if this is
 because we're more strict in our getActorRoom() implementation for v5, or
 if we don't handle these startScript() and Var calls exactly as the
 original does.
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/832#comment:9>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM