[vlc-devel] [PATCH 8/9] video_output: split vout_Request to have a function to enable the vout window

Steve Lhomme robux4 at ycbcr.xyz
Thu Oct 10 16:27:03 CEST 2019


---
 src/video_output/video_output.c | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/video_output/video_output.c b/src/video_output/video_output.c
index dc6b68ac003..08fdc8a80db 100644
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -1944,7 +1944,7 @@ int vout_ChangeSource( vout_thread_t *vout, const video_format_t *original, unsi
     return -1;
 }
 
-int vout_Request(const vout_configuration_t *cfg, input_thread_t *input)
+static int VoutEnableWindow(const vout_configuration_t *cfg)
 {
     vout_thread_t *vout = cfg->vout;
     vout_thread_sys_t *sys = vout->p;
@@ -1952,7 +1952,6 @@ int vout_Request(const vout_configuration_t *cfg, input_thread_t *input)
     assert(!sys->dummy);
     assert(vout != NULL);
     assert(cfg->fmt != NULL);
-    assert(cfg->clock != NULL);
 
     if (!VoutCheckFormat(cfg->fmt))
         return -1;
@@ -1990,20 +1989,33 @@ int vout_Request(const vout_configuration_t *cfg, input_thread_t *input)
 
         if (vout_window_Enable(sys->display_cfg.window, &wcfg)) {
             vlc_mutex_unlock(&sys->window_lock);
+            msg_Err(vout, "window enabling failed");
             video_format_Clean(&sys->original);
             return -1;
         }
         sys->window_enabled = true;
     } else
         vout_UpdateWindowSizeLocked(vout);
+    vlc_mutex_unlock(&sys->window_lock);
+
+    return 0;
+}
+
+int vout_Request(const vout_configuration_t *cfg, input_thread_t *input)
+{
+    vout_thread_t *vout = cfg->vout;
+    vout_thread_sys_t *sys = vout->p;
+
+    assert(cfg->clock != NULL);
+
+    if (VoutEnableWindow(cfg) != 0)
+        return -1;
 
     sys->delay = 0;
     sys->rate = 1.f;
     sys->clock = cfg->clock;
     sys->delay = 0;
 
-    vlc_mutex_unlock(&sys->window_lock);
-
     if (vout_Start(vout, cfg))
     {
         msg_Err(vout, "video output display creation failed");
-- 
2.17.1



More information about the vlc-devel mailing list