[vlc-devel] [PATCH 2/7] win32/inhibit: signal while holding the lock

RĂ©mi Denis-Courmont remi at remlab.net
Mon Feb 3 22:13:13 CET 2020


This is the 21st century. The C run-time threading subsystem knows better
than you how to optimise condition variables.
---
 modules/video_output/win32/inhibit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/video_output/win32/inhibit.c b/modules/video_output/win32/inhibit.c
index d23dcc33e4..566b311189 100644
--- a/modules/video_output/win32/inhibit.c
+++ b/modules/video_output/win32/inhibit.c
@@ -42,8 +42,8 @@ static void Inhibit (vlc_inhibit_t *ih, unsigned mask)
     vlc_inhibit_sys_t *sys = ih->p_sys;
     vlc_mutex_lock(&sys->mutex);
     sys->mask = mask;
-    vlc_mutex_unlock(&sys->mutex);
     vlc_cond_signal(&sys->cond);
+    vlc_mutex_unlock(&sys->mutex);
 }
 
 static void RestoreStateOnCancel( void* p_opaque )
-- 
2.25.0



More information about the vlc-devel mailing list