diff options
author | wm4 <wm4@nowhere> | 2020-02-27 02:33:51 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2020-02-27 02:33:51 +0100 |
commit | c4440db7448ab8b64103af46e7ffb38e3e57bc0e (patch) | |
tree | b5265b4889ff2345646b77dde8bc6e219a5f274e /waftools/dependencies.py | |
parent | 423323170b01794084beb1d9993a10d8da84952b (diff) | |
download | mpv-c4440db7448ab8b64103af46e7ffb38e3e57bc0e.tar.bz2 mpv-c4440db7448ab8b64103af46e7ffb38e3e57bc0e.tar.xz |
sub: do not ignore demuxer wakeups
Setting demux_set_stream_wakeup_cb() will make all sh_stream (i.e.
track) specific wakeups go to this callback. But the callback takes care
of only the sub_preload() case (where it tries to pre-load subtitles
from already parsed and memory-present subtitles in a blocking way).
The old code assumed that the normal demuxer wakeup callback is called.
This was disregarded when the newer code was added. (And actually, the
original plan was to make _all_ per-sh_stream wakeups go to specialized
callbacks to avoid wasted work. dec_sub really should set the callback
always, and propagate wakeups to the playloop code. But it's too far
into the night to write coherent code.)
I couldn't actually observe any manifestation of this bug. Normally, the
playloop wakes up for other reasons (such as driving audio and video
decoding), so the lost wakeups rarely matter.
Diffstat (limited to 'waftools/dependencies.py')
0 files changed, 0 insertions, 0 deletions