[vlc-devel] [PATCH v3 07/18] 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 dbfc21c58b0..b7ae9947b81 100644
--- a/src/video_output/video_output.c
+++ b/src/video_output/video_output.c
@@ -900,11 +900,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;
@@ -959,9 +958,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;
@@ -1017,9 +1013,6 @@ static void ThreadChangeFilters(vout_thread_sys_t *vout,
         free(sys->filter.configuration);
         sys->filter.configuration = filters ? strdup(filters) : NULL;
     }
-
-    if (!is_locked)
-        vlc_mutex_unlock(&sys->filter.lock);
 }
 
 
@@ -1087,7 +1080,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);
                 }
             }
         }
@@ -1438,7 +1431,7 @@ static int ThreadDisplayPicture(vout_thread_sys_t *vout, vlc_tick_t *deadline)
     if (sys->filter.changed ||
         sys->filter.new_interlaced != sys->private.interlacing.has_deint)
     {
-        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