diff options
| author | Carl Hetherington <cth@carlh.net> | 2018-02-18 01:26:03 +0000 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2018-02-18 01:26:03 +0000 |
| commit | e941ab2b02cf01ab15427e47124b7dd32425095c (patch) | |
| tree | 805c5e08a7dda066a60ec0764ae0d3d0ae4d75f1 /cscript | |
| parent | 59500421bda6e44bc6e92e27c6c21ea071eb81c1 (diff) | |
Hold lock for the whole of Butler::video(). Fixes jumpy nudge-seek
when playing DCPs.
Before, the lock was only held to check _pending_seek_position.
With that arrangement, this sequence was possible:
1. video arrives, check _pending_seek_position, decide it's ok, release lock
2. Butler::seek called, sets _pending_seek_position, clears _video
3. video that arrived in #1 is put into _video by ::video()
4. that video is given to an awaiting get_video() call.
This clearly subverts the attempts not to accept new video data while
a seek is pending.
Diffstat (limited to 'cscript')
0 files changed, 0 insertions, 0 deletions
