[Scummvm-tracker] [ScummVM :: Bugs] #16214: SCUMM: MANIAC (v2): Sandy walking animation plays even when still

[ScummVM :: Bugs]

#16214: SCUMM: MANIAC (v2): Sandy walking animation plays even when still
----------------------------------------------+----------------------------
Reporter:  Jibun                              |       Owner:  (none)
    Type:  defect                             |      Status:  new
Priority:  normal                             |   Component:  Engine: SCUMM
 Version:                                     |  Resolution:
Keywords:  Sandy, walking, animation, sprite  |        Game:  Maniac
                                              |  Mansion
----------------------------------------------+----------------------------
Comment (by Jibun):

 Indeed, this error is also present in the original version. I’m not sure
 if it should actually be treated as a bug, then.

 By comparing the `45.LFL` file of the demo and the full release, I’ve
 noticed that only a few bytes differ — and these seem to account for all
 the small differences in the demo sequence.
 For example, the first time Sandy appears and walks (the case where the
 bug occurs), the issue can be fixed by changing just one byte at position
 `11758` within the file.

 If someone knows the exact opcodes and how they work, they might be able
 to determine precisely what the differences are and whether they should be
 fixed and applied to the rest of the game.

 Based on some guesswork and the help of the wiki, I believe that between
 byte `11750` and `11762` the following is executed:
 {{{
 36 17 4D 00 3B 17 11 17 F7 2E F0 FF FF
 }}}

 {{{
 walkActorToObject actor(17) object(4D 00)
 waitForActor actor(17)
 animateActor actor(17) animation(F7) <- here’s the error, in the final
 game this is FF instead of F7
 delay param(F0 FF FF)
 }}}
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/16214#comment:6>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM