From: Rémi Denis-Courmont <rdenis@simphalempin.com>
Date: Sun, 7 Sep 2008 13:16:06 +0000 (+0300)
Subject: Win32: fix vlc_cond_timedwait as well
X-Git-Tag: 1.0.0-pre1~3471
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=d7e6fd9eb47743a6661d6660344f0b01759a4b2b;p=vlc

Win32: fix vlc_cond_timedwait as well
---

diff --git a/include/vlc_threads.h b/include/vlc_threads.h
index 392db06865..7a265b0d4b 100644
--- a/include/vlc_threads.h
+++ b/include/vlc_threads.h
@@ -386,20 +386,14 @@ static inline void __vlc_cond_wait( const char * psz_file, int i_line,
     vlc_mutex_lock( p_mutex );
 
 #elif defined( WIN32 )
-    DWORD result;
-
     do
-    {
         vlc_testcancel ();
-        result = SignalObjectAndWait (*p_mutex, *p_condvar, INFINITE, TRUE);
-    }
-    while (result == WAIT_IO_COMPLETION);
+    while (SignalObjectAndWait (*p_mutex, *p_condvar, INFINITE, TRUE)
+            == WAIT_IO_COMPLETION);
 
     /* Reacquire the mutex before returning. */
     vlc_mutex_lock( p_mutex );
 
-    vlc_testcancel ();
-
     (void)psz_file; (void)i_line;
 
 #endif
@@ -445,25 +439,25 @@ static inline int __vlc_cond_timedwait( const char * psz_file, int i_line,
     return (result == WAIT_TIMEOUT) ? ETIMEDOUT : 0;
 
 #elif defined( WIN32 )
-    mtime_t total;
-    DWORD result = WAIT_TIMEOUT;
+    DWORD result;
 
     (void)psz_file; (void)i_line;
 
-    vlc_testcancel ();
-    while ((total = (deadline - mdate ()) > 0))
+    do
     {
-        DWORD delay = (total > 0x7fffffff) ? 0x7fffffff : total;
-        result = SignalObjectAndWait( *p_mutex, *p_condvar,
-                                      delay, TRUE );
-
-        /* Reacquire the mutex before return/cancel. */
-        vlc_mutex_lock (p_mutex);
-        if (result == WAIT_OBJECT_0)
-            return 0; /* Condition signaled! */
         vlc_testcancel ();
+
+        mtime_t total = deadline - mdate ();
+        if (total <= 0)
+            break;
+        DWORD delay = (total > 0x7fffffff) ? 0x7fffffff : total;
+        result = SignalObjectAndWait (*p_mutex, *p_condvar, delay, TRUE);
     }
-    return ETIMEDOUT;
+    while (result == WAIT_IO_COMPLETION);
+
+    /* Reacquire the mutex before return/cancel. */
+    vlc_mutex_lock (p_mutex);
+    return (result == WAIT_OBJECT_0) ? 0 : ETIMEDOUT;
 
 #endif
 }