From 025999f216f0fe99d0eacd1f7a9c6276b834bcb8 Mon Sep 17 00:00:00 2001
From: Stefan Boberg <stefan@boberg.org>
Date: Fri, 17 Sep 2021 19:04:26 +0200
Subject: Removed WindowsException from public headers

---
 zencore/thread.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'zencore/thread.cpp')

diff --git a/zencore/thread.cpp b/zencore/thread.cpp
index 1c7e4b3ab..c02bf508a 100644
--- a/zencore/thread.cpp
+++ b/zencore/thread.cpp
@@ -257,7 +257,8 @@ ProcessHandle::Wait(int TimeoutMs)
 
 		case WAIT_FAILED:
 			// What might go wrong here, and what is meaningful to act on?
-			throw WindowsException("Process::Wait failed");
+			using namespace std::literals;
+			ThrowLastError("Process::Wait failed"sv);
 	}
 
 	return false;
-- 
cgit v1.2.3


From 28de689b30b64c0c2691b05866c27c84e0a21ff3 Mon Sep 17 00:00:00 2001
From: Stefan Boberg <stefan@boberg.org>
Date: Fri, 17 Sep 2021 23:15:56 +0200
Subject: IsProcessRunning now throws if it fails the function fails to get a
 handle to the process due to an error (unless it is because the process does
 not exist)

---
 zencore/thread.cpp | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

(limited to 'zencore/thread.cpp')

diff --git a/zencore/thread.cpp b/zencore/thread.cpp
index c02bf508a..c92cca6de 100644
--- a/zencore/thread.cpp
+++ b/zencore/thread.cpp
@@ -334,16 +334,32 @@ ProcessMonitor::IsActive() const
 bool
 IsProcessRunning(int pid)
 {
+	// This function is arguably not super useful, a pid can be re-used
+	// by the OS so holding on to a pid and polling it over some time
+	// period will not necessarily tell you what you probably want to know.
+
+#if ZEN_PLATFORM_WINDOWS
 	HANDLE hProc = OpenProcess(PROCESS_QUERY_LIMITED_INFORMATION, FALSE, pid);
 
-	if (hProc == NULL)
+	if (!hProc)
 	{
-		return false;
+		DWORD Error = zen::GetLastError();
+
+		if (Error == ERROR_INVALID_PARAMETER)
+		{
+			return false;
+		}
+
+		using namespace fmt::literals;
+		ThrowSystemError(Error, "failed to open process with pid {}"_format(pid));
 	}
 
 	CloseHandle(hProc);
 
 	return true;
+#else
+	ZEN_NOT_IMPLEMENTED();
+#endif
 }
 
 int
-- 
cgit v1.2.3


From c86315ea955408659eb7ea32798693975e27b9b7 Mon Sep 17 00:00:00 2001
From: Stefan Boberg <stefan@boberg.org>
Date: Sun, 19 Sep 2021 22:35:12 +0200
Subject: Changed so Windows also uses the portable std::mutex implementation
 and reworked some code which would not compile after the change

---
 zencore/thread.cpp | 16 ----------------
 1 file changed, 16 deletions(-)

(limited to 'zencore/thread.cpp')

diff --git a/zencore/thread.cpp b/zencore/thread.cpp
index c92cca6de..d4f101454 100644
--- a/zencore/thread.cpp
+++ b/zencore/thread.cpp
@@ -17,41 +17,25 @@ namespace zen {
 void
 RwLock::AcquireShared()
 {
-#if ZEN_PLATFORM_WINDOWS
-	AcquireSRWLockShared((PSRWLOCK)&m_Srw);
-#else
 	m_Mutex.lock_shared();
-#endif
 }
 
 void
 RwLock::ReleaseShared()
 {
-#if ZEN_PLATFORM_WINDOWS
-	ReleaseSRWLockShared((PSRWLOCK)&m_Srw);
-#else
 	m_Mutex.unlock_shared();
-#endif
 }
 
 void
 RwLock::AcquireExclusive()
 {
-#if ZEN_PLATFORM_WINDOWS
-	AcquireSRWLockExclusive((PSRWLOCK)&m_Srw);
-#else
 	m_Mutex.lock();
-#endif
 }
 
 void
 RwLock::ReleaseExclusive()
 {
-#if ZEN_PLATFORM_WINDOWS
-	ReleaseSRWLockExclusive((PSRWLOCK)&m_Srw);
-#else
 	m_Mutex.unlock();
-#endif
 }
 
 Event::Event()
-- 
cgit v1.2.3