[vlc-commits] [Git][videolan/vlc][master] video_output: rename confusing internal variables

[Jean-Baptiste Kempf (@jbk)]

Jean-Baptiste Kempf pushed to branch master at VideoLAN / VLC


Commits:
a09a26d4 by Steve Lhomme at 2021-09-15T07:46:36+00:00
video_output: rename confusing internal variables

The `next_system_pts` has nothing to do with the displayed.next picture. It's
the date of the displayed.current picture in system time.

The `date_next` also has nothing to do with the displayed.next picture. It's
the system date where we should initiate the prepare() for displayed.next.

- - - - -


1 changed file:

- src/video_output/video_output.c


Changes:

=====================================
src/video_output/video_output.c
=====================================
@@ -1474,13 +1474,13 @@ static int DisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
     }
     else if (!paused)
     {
-        const vlc_tick_t next_system_pts =
+        const vlc_tick_t system_swap_current =
             vlc_clock_ConvertToSystem(sys->clock, system_now,
                                       sys->displayed.current->date, sys->rate);
-        if (likely(next_system_pts != VLC_TICK_MAX))
+        if (likely(system_swap_current != VLC_TICK_MAX))
         {
-            vlc_tick_t date_next = next_system_pts - render_delay;
-            if (date_next <= system_now)
+            vlc_tick_t system_prepare_current = system_swap_current - render_delay;
+            if (system_prepare_current <= system_now)
             {
                 // the current frame will be late, look for the next not late one
                 next = PreparePicture(vout, false, false);



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/a09a26d4b7fa788f057b43f6022ba99bdee9b6ec

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/a09a26d4b7fa788f057b43f6022ba99bdee9b6ec
You're receiving this email because of your account on code.videolan.org.