From b87ce0d8e30f4b2da7d643c4e2550c22817aeece Mon Sep 17 00:00:00 2001
From: Steve Lhomme <robux4@ycbcr.xyz>
Date: Fri, 16 Oct 2020 14:54:48 +0200
Subject: [PATCH] video_output: compute force_refresh earlier

First if a re-display is needed. And then if the current frame has changed it
doesn't need to be displayed ASAP, we know it's within the render delay.

Merge force_refresh and render_now since they mean the same thing.
---
 src/video_output/video_output.c | 54 ++++++++++++++++-----------------
 1 file changed, 27 insertions(+), 27 deletions(-)

diff --git a/src/video_output/video_output.c b/src/video_output/video_output.c
index dbae199289..57ee54e6e8 100644
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -1484,7 +1484,7 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
             return VLC_EGENERIC; // wait with no known deadline
     }
 
-    bool force_refresh;
+    bool render_now;
     if (frame_by_frame)
     {
         if (!sys->displayed.next)
@@ -1497,7 +1497,7 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
         sys->displayed.current = sys->displayed.next;
         sys->displayed.next    = NULL;
 
-        force_refresh = true;
+        render_now = true;
     }
     else
     {
@@ -1514,7 +1514,24 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
         const vlc_tick_t render_delay = vout_chrono_GetHigh(&sys->render) + VOUT_MWAIT_TOLERANCE;
 
         bool dropped_current_frame = false;
-        vlc_tick_t date_next = VLC_TICK_INVALID;
+
+        /* FIXME/XXX we must redisplay the last decoded picture (because
+        * of potential vout updated, or filters update or SPU update)
+        * For now a high update period is needed but it could be removed
+        * if and only if:
+        * - vout module emits events from themselves.
+        * - *and* SPU is modified to emit an event or a deadline when needed.
+        *
+        * So it will be done later.
+        */
+        bool refresh = false;
+
+        vlc_tick_t date_refresh = VLC_TICK_INVALID;
+        if (sys->displayed.date != VLC_TICK_INVALID) {
+            date_refresh = sys->displayed.date + VOUT_REDISPLAY_DELAY - render_delay;
+            refresh = date_refresh <= system_now;
+        }
+        render_now = refresh;
 
         if (!paused && sys->displayed.next) {
             const vlc_tick_t next_system_pts =
@@ -1522,11 +1539,15 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
                                         sys->displayed.next->date, sys->rate);
             if (likely(next_system_pts != INT64_MAX))
             {
-                date_next = next_system_pts - render_delay;
+                vlc_tick_t date_next = next_system_pts - render_delay;
+                if (date_refresh == VLC_TICK_INVALID || date_next < date_refresh)
+                    date_refresh = date_next;
+
                 if (date_next <= system_now)
                 {
                     // next frame will still need some waiting before display
                     dropped_current_frame = true;
+                    render_now = false;
 
                     picture_Release(sys->displayed.current);
                     sys->displayed.current = sys->displayed.next;
@@ -1534,29 +1555,8 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
                 }
             }
         }
-
-        /* FIXME/XXX we must redisplay the last decoded picture (because
-        * of potential vout updated, or filters update or SPU update)
-        * For now a high update period is needed but it could be removed
-        * if and only if:
-        * - vout module emits events from theselves.
-        * - *and* SPU is modified to emit an event or a deadline when needed.
-        *
-        * So it will be done later.
-        */
-        bool refresh = false;
-
-        vlc_tick_t date_refresh = VLC_TICK_INVALID;
-        if (sys->displayed.date != VLC_TICK_INVALID) {
-            date_refresh = sys->displayed.date + VOUT_REDISPLAY_DELAY - render_delay;
-            refresh = date_refresh <= system_now;
-        }
-        force_refresh = !dropped_current_frame && refresh;
-
         if (date_refresh != VLC_TICK_INVALID)
             *deadline = date_refresh;
-        if (date_next != VLC_TICK_INVALID && date_next < *deadline)
-            *deadline = date_next;
 
         if (!first && !refresh && !dropped_current_frame) {
             // nothing changed, wait until the next deadline or a control
@@ -1568,9 +1568,9 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
         return VLC_EGENERIC;
 
     /* display the picture immediately */
-    bool render_now = force_refresh || sys->displayed.current->b_force;
+    render_now |= sys->displayed.current->b_force;
     int ret = ThreadDisplayRenderPicture(vout, render_now);
-    return force_refresh ? VLC_EGENERIC : ret;
+    return render_now ? VLC_EGENERIC : ret;
 }
 
 void vout_ChangePause(vout_thread_t *vout, bool is_paused, vlc_tick_t date)