From 4d2b3fca3b5c6e61d2ff3a22f78b1b640500808e Mon Sep 17 00:00:00 2001 From: wm4 Date: Thu, 17 May 2018 13:13:00 +0200 Subject: player: don't reset last_seek_pts on playback state reset This is nonsense. Didn't matter in most situations, because seeking itself set this after it was cleared. But some callers don't do this, see e.g. commit ed73ba89644fc6. There is no need to clear it at all, and it causes issues with the next commit. It only needs to be reset on loading. Also move the initialization on loading up, which doesn't change behavior, but makes the intention clearer. --- player/playloop.c | 1 - 1 file changed, 1 deletion(-) (limited to 'player/playloop.c') diff --git a/player/playloop.c b/player/playloop.c index a784bfd554..f51ddb5e5c 100644 --- a/player/playloop.c +++ b/player/playloop.c @@ -228,7 +228,6 @@ void reset_playback_state(struct MPContext *mpctx) mpctx->hrseek_backstep = false; mpctx->current_seek = (struct seek_params){0}; mpctx->playback_pts = MP_NOPTS_VALUE; - mpctx->last_seek_pts = MP_NOPTS_VALUE; mpctx->step_frames = 0; mpctx->ab_loop_clip = true; mpctx->restart_complete = false; -- cgit v1.2.3