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;
 
 }