[ scummvm-Patches-674566 ] CMI: Further smush syncing

[SourceForge.net]

Patches item #674566, was opened at 2003-01-25 14:32
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=674566&group_id=37116

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Torbjörn Andersson (eriktorbjorn)
Assigned to: Nobody/Anonymous (nobody)
Summary: CMI: Further smush syncing

Initial Comment:
Even with my previous sync patch, the opening video
would get out of sync for me. It turned out that
ScummRenderer::update() would occasionally get called
with _wait already being set to false. As far as I can
understand, this can only mean that my computer wasn't
fast enough to finish drawing the previous frame before
being told to draw the next one.

At 12 frames per second, it doesn't take many missed
frames before it becomes quite noticeable.

For me, this happens occasionally even under normal
load. I made my first experiments with smush syncing
early in the morning, while my computer was running a
number of disk-intensive cron jobs, which probably
explains why I couldn't get it to work right then.

With this patch, the smush player will keep track of
the number of pending updates instead, so that it can
skip frames to "catch up". It works perfectly for me,
but I guess that with a really slow computer you
wouldn't see any frames at all. Maybe it should still
draw the occasional frame, even if it's lagging behind?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=674566&group_id=37116