subtitles: Fix recent filter-rendered libass timing problem
commit55f94a1a91bf9bf28b5b33f698c28a86c08e0e75
authorUoti Urpala <uau@glyph.nonexistent.invalid>
Wed, 2 Dec 2009 15:57:47 +0000 (2 17:57 +0200)
committerUoti Urpala <uau@glyph.nonexistent.invalid>
Wed, 2 Dec 2009 15:57:47 +0000 (2 17:57 +0200)
tree1cf609022ba17dda11cf467eb337147b08bac1a6
parent9afcdab6944587aaee7db0307f3087b5a275771d
subtitles: Fix recent filter-rendered libass timing problem

commit 4c552b2e420ba4cb6d888b12360c7bf63e7cd03a ("core: Do
OSD/subtitle updates at a more accurate point") made filter-rendered
libass subtitles appear one frame too late the first time they became
visible (VO rendering was not affected, and filter rendering was
accurate if seeking back later). The reason was that the subtitle code
did not feed the subtitle events to libass before their starting time,
and this now happened after the filtering phase. Fix this by skipping
the time check in the libass case and feeding demuxed subtitles to
it unconditionally (as timing is done separately anyway with libass).

There are still at least theoretically possible new problems in the
filter-rendered case because the filter now relies on packets demuxed
before the _previous_ frame. There's a bigger chance of getting the
subtitle packet too late, and the filter can't see packets for the
first frame after a seek. However the former is not an issue for the
samples I tested even with -nosound, and subtitles are not in general
guaranteed to be shown when seeking to a new position (though it could
be worth a later improvement).
mpcommon.c