[vlc-devel] [PATCH v2 08/18] video_output: do not use the frame skipping mode if no frame was displayed

[Steve Lhomme]

The frame by frame forces display_next_frame which skips the current frame. If
we didn't have a picture we should display displayed.current, not
displayed.next. So we shouldn't set display_next_frame in that case.
---
 src/video_output/video_output.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/video_output/video_output.c b/src/video_output/video_output.c
index 594d3c9c194..0c193ea233a 100644
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -1500,7 +1500,7 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
     const vlc_tick_t system_now = vlc_tick_now();
     const vlc_tick_t render_delay = vout_chrono_GetHigh(&sys->render) + VOUT_MWAIT_TOLERANCE;
 
-    bool display_next_frame = frame_by_frame;
+    bool display_next_frame = frame_by_frame && !first;
 
     /* FIXME/XXX we must redisplay the last decoded picture (because
      * of potential vout updated, or filters update or SPU update)
-- 
2.26.2