From cb642e7c846be2540ec5c88c854682fc26d787f8 Mon Sep 17 00:00:00 2001 From: wm4 Date: Wed, 27 Aug 2014 22:51:06 +0200 Subject: player: slightly better cache underrun detection Use the "native" underrun detection, instead of guessing by a low cache duration. The new underrun detection (which was added with the original commit) might have the problem that it's easy for the playloop to miss the underrun event. The underrun is actually not stored as state, so if the demuxer thread adds a new packet before the playloop happens to see the state, it's as if it never happened. On the other hand, this means that network was fast enough, so it should be just fine. Also, should it happen that we don't know the cached range (the ts_duration < 0 case), just wait until the demuxer goes idle (i.e. read_packet() decides to stop). This pretty much should affect broken or unusual files only, and there might be various things that could go wrong. But it's more robust in the normal case: this situation also happens when no packets have been read yet, and we don't want to consider this as reason to resume playback. --- demux/demux.c | 1 + 1 file changed, 1 insertion(+) (limited to 'demux/demux.c') diff --git a/demux/demux.c b/demux/demux.c index 8ddfd3f89c..6b4f25be51 100644 --- a/demux/demux.c +++ b/demux/demux.c @@ -1187,6 +1187,7 @@ static int cached_demux_control(struct demux_internal *in, int cmd, void *arg) } } r->idle = (in->idle && !r->underrun) || r->eof; + r->underrun &= !r->idle; if (r->ts_range[0] != MP_NOPTS_VALUE && r->ts_range[1] != MP_NOPTS_VALUE) r->ts_duration = r->ts_range[1] - r->ts_range[0]; return DEMUXER_CTRL_OK; -- cgit v1.2.3