[vlc-devel] [PATCH v5 11/24] video_output: remove always true boolean from ThreadChangeFilters

[Steve Lhomme]

We always call this function under filter lock.
---
 src/video_output/video_output.c | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/src/video_output/video_output.c b/src/video_output/video_output.c
index acb3c2cc8b4..352006247d8 100644
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -922,11 +922,10 @@ typedef struct {
 
 static void ThreadChangeFilters(vout_thread_sys_t *vout,
                                 const char *filters,
-                                const bool *new_deinterlace,
-                                bool is_locked)
+                                const bool *new_deinterlace)
 {
     vout_thread_sys_t *sys = vout;
-    ThreadFilterFlush(vout, is_locked);
+    ThreadFilterFlush(vout, true);
     ThreadDelAllFilterCallbacks(vout);
 
     vlc_array_t array_static;
@@ -981,9 +980,6 @@ static void ThreadChangeFilters(vout_thread_sys_t *vout,
         current = next;
     }
 
-    if (!is_locked)
-        vlc_mutex_lock(&sys->filter.lock);
-
     es_format_t fmt_target;
     es_format_InitFromVideo(&fmt_target, &sys->filter.src_fmt);
     vlc_video_context *vctx_target  = sys->filter.src_vctx;
@@ -1041,9 +1037,6 @@ static void ThreadChangeFilters(vout_thread_sys_t *vout,
     }
 
     sys->filter.changed = false;
-
-    if (!is_locked)
-        vlc_mutex_unlock(&sys->filter.lock);
 }
 
 
@@ -1111,7 +1104,7 @@ static int ThreadDisplayPreparePicture(vout_thread_sys_t *vout, bool reuse,
                         vlc_video_context_Release(sys->filter.src_vctx);
                     sys->filter.src_vctx = pic_vctx ? vlc_video_context_Hold(pic_vctx) : NULL;
 
-                    ThreadChangeFilters(vout, NULL, NULL, true);
+                    ThreadChangeFilters(vout, NULL, NULL);
                 }
             }
         }
@@ -1460,7 +1453,7 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
     if (sys->filter.changed ||
         sys->private.interlacing.has_deint != sys->filter.new_interlaced)
     {
-        ThreadChangeFilters(vout, sys->filter.new_filters, &sys->filter.new_interlaced, true);
+        ThreadChangeFilters(vout, sys->filter.new_filters, &sys->filter.new_interlaced);
     }
     vlc_mutex_unlock(&sys->filter.lock);
 
-- 
2.26.2