[Scummvm-tracker] [ScummVM :: Bugs] #15303: SCI: LSL5: Timed message skipped during LSL5 coffee scene

[ScummVM :: Bugs]

#15303: SCI: LSL5: Timed message skipped during LSL5 coffee scene
-------------------------+-----------------------------------
Reporter:  eriktorbjorn  |       Owner:  (none)
    Type:  defect        |      Status:  new
Priority:  normal        |   Component:  Engine: SCI
 Version:                |  Resolution:
Keywords:                |        Game:  Leisure Suit Larry 5
-------------------------+-----------------------------------
Comment (by eriktorbjorn):

 But if I'm even a few seconds off, it probably won't happen. I tried a few
 times in ScummVM to no avail before filing this bug report, but had to
 resort to extracting the savegame from my phone, so that I could work with
 that until I had a savegame that reproduces the problem every time for me.

 This seems to be the part that updates `ticks`, retrieved with "disasm
 Script doit bc":

 {{{
 0004:06dd: 63 16          pToa  ticks[16]
 0004:06df: 30 1f 00       bnt   001f  [0701]
 0004:06e2: 36             push
 0004:06e3: 78             push1                 ; superClass
 0004:06e4: 89 58          lsg   58              ; 88, 'X'
 0004:06e6: 63 18          pToa  lastTicks[18]
 0004:06e8: 04             sub
 0004:06e9: 36             push
 0004:06ea: 43 3d 02       callk Abs[3d],        02
 0004:06ed: 04             sub
 0004:06ee: 65 16          aTop  ticks[16]
 0004:06f0: 36             push
 0004:06f1: 35 00          ldi   00
 0004:06f3: 24             le?
 0004:06f4: 30 0a 00       bnt   000a  [0701]
 0004:06f7: 35 00          ldi   00
 0004:06f9: 65 16          aTop  ticks[16]
 0004:06fb: 38 8d 00       pushi 008d            ; 141, cue
 0004:06fe: 76             push0                 ; species
 0004:06ff: 54 04          self  04
 }}}

 But I'm still not sure what happens here. I thought it was that integer
 overflow caused some large negative value to be passed to `kAbs` but that
 doesn't seem to be the case. There were a couple of large - but not
 ''that'' large - positive values, but I don't know if they came from this
 script or not.
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/15303#comment:10>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM