From bfa6c9ab1e2f316c53811de6e311212b8a088591 Mon Sep 17 00:00:00 2001
From: Tannin <devnull@localhost>
Date: Mon, 29 Sep 2014 20:35:35 +0200
Subject: - moved the hook-recursion-protection to tls - some code cleanup and
 consolidation - hook.dll will now report all of its own exceptions - some
 more logging during startup - changed the way urls are encoded for download
 requests - now displaying (one of the) process name(s) while waiting for a
 program to end - bugfix: spawned processes were forced to leave the job

---
 src/mainwindow.cpp | 49 +++++++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 45 insertions(+), 4 deletions(-)

(limited to 'src/mainwindow.cpp')

diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index 5feda937..20738a99 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -1405,6 +1405,15 @@ HANDLE MainWindow::spawnBinaryDirect(const QFileInfo &binary, const QString &arg
   }
 }
 
+std::wstring getProcessName(DWORD processId)
+{
+  HANDLE process = ::OpenProcess(PROCESS_QUERY_INFORMATION, false, processId);
+
+  DWORD value = MAX_PATH;
+  wchar_t buffer[MAX_PATH];
+  ::QueryFullProcessImageNameW(process, 0, buffer, &value);
+  return buffer;
+}
 
 void MainWindow::spawnBinary(const QFileInfo &binary, const QString &arguments, const QDir &currentDirectory, bool closeAfterStart, const QString &steamAppID)
 {
@@ -1423,11 +1432,43 @@ void MainWindow::spawnBinary(const QFileInfo &binary, const QString &arguments,
 
       QCoreApplication::processEvents();
 
-      while ((::WaitForSingleObject(processHandle, 100) == WAIT_TIMEOUT) &&
-              !dialog->unlockClicked()) {
-        // keep processing events so the app doesn't appear dead
-        QCoreApplication::processEvents();
+      DWORD retLen;
+      JOBOBJECT_BASIC_PROCESS_ID_LIST info;
+
+      {
+        DWORD currentProcess = 0UL;
+        bool isJobHandle = true;
+
+        DWORD res = ::MsgWaitForMultipleObjects(1, &processHandle, false, 1000, QS_KEY | QS_MOUSE);
+        while ((res != WAIT_FAILED) && (res != WAIT_OBJECT_0) && !dialog->unlockClicked()) {
+          if (isJobHandle) {
+            if (::QueryInformationJobObject(processHandle, JobObjectBasicProcessIdList, &info, sizeof(info), &retLen) > 0) {
+              if (info.NumberOfProcessIdsInList == 0) {
+              } else {
+                if (info.ProcessIdList[0] != currentProcess) {
+                  currentProcess = info.ProcessIdList[0];
+                  dialog->setProcessName(ToQString(getProcessName(currentProcess)));
+                }
+                break;
+              }
+            } else {
+              // the info-object I passed only provides space for 1 process id. but since this code only cares about whether there
+              // is more than one that's good enough. ERROR_MORE_DATA simply signals there are at least two processes running.
+              // any other error probably means the handle is a regular process handle, probably caused by running MO in a job without
+              // the right to break out.
+              if (::GetLastError() != ERROR_MORE_DATA) {
+                isJobHandle = false;
+              }
+            }
+          }
+
+          // keep processing events so the app doesn't appear dead
+          QCoreApplication::processEvents();
+
+          res = ::MsgWaitForMultipleObjects(1, &processHandle, false, 1000, QS_KEY | QS_MOUSE);
+        }
       }
+      ::CloseHandle(processHandle);
 
       this->setEnabled(true);
       refreshDirectoryStructure();
-- 
cgit v1.3.1