You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ta...@apache.org on 2012/04/09 00:52:45 UTC

svn commit: r1311114 - /activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/windows/PlatformThread.cpp

Author: tabish
Date: Sun Apr  8 22:52:45 2012
New Revision: 1311114

URL: http://svn.apache.org/viewvc?rev=1311114&view=rev
Log:
Unlock of platform RW Mutex was incorrectly throwing an exception for unlock of the write lock

Modified:
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/windows/PlatformThread.cpp

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/windows/PlatformThread.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/windows/PlatformThread.cpp?rev=1311114&r1=1311113&r2=1311114&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/windows/PlatformThread.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/windows/PlatformThread.cpp Sun Apr  8 22:52:45 2012
@@ -209,7 +209,7 @@ void PlatformThread::unlockRWMutex(decaf
                    __FILE__, __LINE__, "Failed to signal OS event object.");
            }
         }
-    } else {
+    } else if (result != 0) {
         throw RuntimeException(
             __FILE__, __LINE__, "Failed to signal unlock OS Mutex object.");
     }
@@ -227,7 +227,7 @@ void PlatformThread::destroyRWMutex(deca
 ////////////////////////////////////////////////////////////////////////////////
 void PlatformThread::createCondition(decaf_condition_t* condition) {
 
-    *condition = ::CreateEvent(NULL, TRUE, FALSE, NULL);
+    *condition = ::CreateEvent(NULL, FALSE, FALSE, NULL);
 
     if (condition == NULL) {
         throw RuntimeException(