[vlc-commits] adaptive: PlaylistManager: remove useless mutex lock

Alexandre Janniaux git at videolan.org
Thu Nov 12 14:15:21 CET 2020


vlc | branch: master | Alexandre Janniaux <ajanni at videolabs.io> | Sun Nov  1 12:34:19 2020 +0100| [551734e0ac03308560c402626d83eb99ba356bb9] | committer: Alexandre Janniaux

adaptive: PlaylistManager: remove useless mutex lock

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=551734e0ac03308560c402626d83eb99ba356bb9
---

 modules/demux/adaptive/PlaylistManager.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules/demux/adaptive/PlaylistManager.cpp b/modules/demux/adaptive/PlaylistManager.cpp
index 1f0ecbec4b..c49c748293 100644
--- a/modules/demux/adaptive/PlaylistManager.cpp
+++ b/modules/demux/adaptive/PlaylistManager.cpp
@@ -670,9 +670,9 @@ void PlaylistManager::Run()
             else /*if(i_return == AbstractStream::buffering_suspended)*/
                 i_deadline += VLC_TICK_FROM_MS(250);
 
-            vlc_mutex_lock(&demux.lock);
+            // TODO: The current function doesn't seem to modify shared
+            //       state under demux lock.
             vlc_cond_signal(&demux.cond);
-            vlc_mutex_unlock(&demux.lock);
 
             while(b_buffering &&
                     vlc_cond_timedwait(&waitcond, &lock, i_deadline) == 0 &&



More information about the vlc-commits mailing list