Merge pull request #72 from sjanuary/winfix

Fix Semaphore bug on Windows (no CPU/mem/env data on Node 10)
RuntimeTools · Nov 21, 2018 · 918dbd3 · 918dbd3
2 parents d94245a + 398c6aa
commit 918dbd3
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/src/ibmras/common/port/windows/Thread.cpp b/src/ibmras/common/port/windows/Thread.cpp
@@ -85,9 +85,19 @@ bool Semaphore::wait(uint32 timeout) {
 
 	IBMRAS_DEBUG(finest,  "Semaphore::wait");
 	DWORD retVal = WaitForSingleObject(handle, timeout * 1000);
-	if ( !GetLastError()) {
-		return (retVal == WAIT_OBJECT_0);
-	}
+	switch (retVal) 
+        { 
+            case WAIT_OBJECT_0: 
+		IBMRAS_DEBUG(finest,  "Semaphore::wait SUCCESS");
+                return true;
+
+            case WAIT_TIMEOUT: 
+		IBMRAS_DEBUG(finest,  "Semaphore::wait WAIT_TIMEOUT");
+                break; 
+
+	    case WAIT_FAILED:
+		IBMRAS_DEBUG(finest,  "Semaphore::wait WAIT_FAILED");
+	}	
 	return false;
 
 }