summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2015-04-28 22:03:18 +0200
committerwm4 <wm4@nowhere>2015-04-28 22:03:18 +0200
commit0ff93a83571ede54af87ebd1aed5736f428c48d4 (patch)
tree3af14ce7c62a967cdfdd2630f068b234f4ef3986
parentc4aa13615501189c55c23448d436074e5f92c8cc (diff)
downloadmpv-0ff93a83571ede54af87ebd1aed5736f428c48d4.tar.bz2
mpv-0ff93a83571ede54af87ebd1aed5736f428c48d4.tar.xz
player: clamp display time to known time range on seeking
During seeking, and there is momemtarily no new data available yet, the player will display the seek target as current time. Clamp this time to the known time range as implied by the start time and the duration of the file. This improves behavior especially when seeking in audio files, for which this for some reason triggers rather often. There were some users complaining about this. This makes behavior worse for files with timestamp resets, or incorrectly reported duration. (The latter is relatively common, e.g. libavformat shortcomings, or incomplete files.)
-rw-r--r--player/playloop.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/player/playloop.c b/player/playloop.c
index c34ae7e6da..0c0de44997 100644
--- a/player/playloop.c
+++ b/player/playloop.c
@@ -382,6 +382,13 @@ double get_playback_time(struct MPContext *mpctx)
{
double cur = get_current_time(mpctx);
double start = get_start_time(mpctx);
+ // During seeking, the time corresponds to the last seek time - apply some
+ // cosmetics to it.
+ if (mpctx->playback_pts == MP_NOPTS_VALUE) {
+ double length = get_time_length(mpctx);
+ if (length >= 0)
+ cur = MPCLAMP(cur, start, start + length);
+ }
return cur >= start ? cur - start : cur;
}