player: respect --untimed on last frame

The last video frame is another case that has a separate code path,
although it's pretty similar to the one in commit 73e5aa87. Fix this
in a different way, which also takes care of the last frame case,
although without context the code becomes slightly more tricky.

As further cleanup, move the decision about framedropping itself to
the same place, so the check in vo.c becomes much simpler. The check
for the vo->driver->encode flag, which is remvoed completely, was
redundant too.

Fixes #1480.

2 files changed, 4 insertions, 3 deletions

diff --git a/player/video.c b/player/video.c
index 50fa8d7a3d..55daade37a 100644
--- a/player/video.c
+++ b/player/video.c
@@ -756,7 +756,7 @@ void write_video(struct MPContext *mpctx, double endpts)
     struct mp_image_params p = mpctx->next_frame[0]->params;
     if (!vo->params || !mp_image_params_equal(&p, vo->params)) {
         // Changing config deletes the current frame; wait until it's finished.
-        if (vo_still_displaying(vo) && !(opts->untimed || vo->driver->untimed))
+        if (vo_still_displaying(vo))
             return;
 
         const struct vo_driver *info = mpctx->video_out->driver;
@@ -790,6 +790,8 @@ void write_video(struct MPContext *mpctx, double endpts)
         diff = vpts1 - vpts0;
     if (diff < 0 && mpctx->d_video->fps > 0)
         diff = 1.0 / mpctx->d_video->fps; // fallback to demuxer-reported fps
+    if (opts->untimed || vo->driver->untimed || !(opts->frame_dropping & 1))
+        diff = -1; // disable frame dropping and aspects of frame timing
     if (diff >= 0) {
         // expected A/V sync correction is ignored
         diff /= opts->playback_speed;
diff --git a/video/out/vo.c b/video/out/vo.c
index 7e2bcfb83e..9c5514ee4f 100644
--- a/video/out/vo.c
+++ b/video/out/vo.c
@@ -561,8 +561,7 @@ static bool render_frame(struct vo *vo)
     int64_t next_vsync = prev_sync(vo, mp_time_us()) + in->vsync_interval;
     int64_t end_time = pts + duration;
 
-    if (!(vo->global->opts->frame_dropping & 1) || !in->hasframe_rendered ||
-        vo->driver->untimed || vo->driver->encode)
+    if (!in->hasframe_rendered)
         duration = -1; // disable framedrop
 
     in->dropped_frame = duration >= 0 && end_time < next_vsync;