From: Mikhail Glushenkov <foldr@codedgers.com>
Date: Tue, 15 Sep 2009 03:39:45 +0000 (+0000)
Subject: Get rid of GetProcessId in Win32/Program.inc.
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=6db0e3a98198fb47b11314935e1201f7ebf2c52d;p=oota-llvm.git

Get rid of GetProcessId in Win32/Program.inc.

GetProcessId was introduced only in XP. As a bonus, this change makes Program
objects copyable, since Program is now basically a PID.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@81826 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/include/llvm/System/Program.h b/include/llvm/System/Program.h
index 67995627260..b200eddc224 100644
--- a/include/llvm/System/Program.h
+++ b/include/llvm/System/Program.h
@@ -29,22 +29,18 @@ namespace sys {
   /// @since 1.4
   /// @brief An abstraction for finding and executing programs.
   class Program {
-    /// Opaque handle for target specific data.
-    void *Data_;
 
-    // Noncopyable.
-    Program(const Program& other);
-    Program& operator=(const Program& other);
+    unsigned Pid_;
 
     /// @name Methods
     /// @{
   public:
 
-    Program();
-    ~Program();
+    Program() : Pid_(0) {}
+    ~Program() {}
 
     /// Return process ID of this program.
-    unsigned GetPid() const;
+    unsigned GetPid() const { return Pid_; }
 
     /// This function executes the program using the \p arguments provided.  The
     /// invoked program will inherit the stdin, stdout, and stderr file
diff --git a/lib/System/Unix/Program.inc b/lib/System/Unix/Program.inc
index 56dea250a77..cf6876a54b1 100644
--- a/lib/System/Unix/Program.inc
+++ b/lib/System/Unix/Program.inc
@@ -34,15 +34,6 @@
 namespace llvm {
 using namespace sys;
 
-Program::Program() : Data_(0) {}
-
-Program::~Program() {}
-
-unsigned Program::GetPid() const {
-  uint64_t pid = reinterpret_cast<uint64_t>(Data_);
-  return static_cast<unsigned>(pid);
-}
-
 // This function just uses the PATH environment variable to find the program.
 Path
 Program::FindProgramByName(const std::string& progName) {
@@ -214,7 +205,7 @@ Program::Execute(const Path& path,
       break;
   }
 
-  Data_ = reinterpret_cast<void*>(child);
+  Pid_ = child;
 
   return true;
 }
@@ -226,7 +217,7 @@ Program::Wait(unsigned secondsToWait,
 #ifdef HAVE_SYS_WAIT_H
   struct sigaction Act, Old;
 
-  if (Data_ == 0) {
+  if (Pid_ == 0) {
     MakeErrMsg(ErrMsg, "Process not started!");
     return -1;
   }
@@ -242,8 +233,7 @@ Program::Wait(unsigned secondsToWait,
 
   // Parent process: Wait for the child process to terminate.
   int status;
-  uint64_t pid = reinterpret_cast<uint64_t>(Data_);
-  pid_t child = static_cast<pid_t>(pid);
+  pid_t child = Pid_;
   while (wait(&status) != child)
     if (secondsToWait && errno == EINTR) {
       // Kill the child.
@@ -291,15 +281,12 @@ Program::Wait(unsigned secondsToWait,
 
 bool
 Program::Kill(std::string* ErrMsg) {
-  if (Data_ == 0) {
+  if (Pid_ == 0) {
     MakeErrMsg(ErrMsg, "Process not started!");
     return true;
   }
 
-  uint64_t pid64 = reinterpret_cast<uint64_t>(Data_);
-  pid_t pid = static_cast<pid_t>(pid64);
-
-  if (kill(pid, SIGKILL) != 0) {
+  if (kill(Pid_, SIGKILL) != 0) {
     MakeErrMsg(ErrMsg, "The process couldn't be killed!");
     return true;
   }
diff --git a/lib/System/Win32/Program.inc b/lib/System/Win32/Program.inc
index af6cce6de1a..f1faff0ed11 100644
--- a/lib/System/Win32/Program.inc
+++ b/lib/System/Win32/Program.inc
@@ -25,21 +25,6 @@
 namespace llvm {
 using namespace sys;
 
-Program::Program() : Data_(0) {}
-
-Program::~Program() {
-  if (Data_) {
-    HANDLE hProcess = reinterpret_cast<HANDLE>(Data_);
-    CloseHandle(hProcess);
-    Data_ = 0;
-  }
-}
-
-unsigned Program::GetPid() const {
-  HANDLE hProcess = reinterpret_cast<HANDLE>(Data_);
-  return GetProcessId(hProcess);
-}
-
 // This function just uses the PATH environment variable to find the program.
 Path
 Program::FindProgramByName(const std::string& progName) {
@@ -137,11 +122,6 @@ Program::Execute(const Path& path,
                  const Path** redirects,
                  unsigned memoryLimit,
                  std::string* ErrMsg) {
-  if (Data_) {
-    HANDLE hProcess = reinterpret_cast<HANDLE>(Data_);
-    CloseHandle(Data_);
-    Data_ = 0;
-  }
 
   if (!path.canExecute()) {
     if (ErrMsg)
@@ -269,9 +249,10 @@ Program::Execute(const Path& path,
                path.str() + "'");
     return false;
   }
-  Data_ = reinterpret_cast<void*>(pi.hProcess);
+  Pid_ = pi.dwProcessId;
 
   // Make sure these get closed no matter what.
+  AutoHandle hProcess(pi.hProcess);
   AutoHandle hThread(pi.hThread);
 
   // Assign the process to a job if a memory limit is defined.
@@ -305,12 +286,17 @@ Program::Execute(const Path& path,
 int
 Program::Wait(unsigned secondsToWait,
               std::string* ErrMsg) {
-  if (Data_ == 0) {
+  if (Pid_ == 0) {
     MakeErrMsg(ErrMsg, "Process not started!");
     return -1;
   }
 
-  HANDLE hProcess = reinterpret_cast<HANDLE>(Data_);
+  HANDLE hOpen = OpenProcess(SYNCHRONIZE, FALSE, Pid_);
+  if (hOpen == NULL) {
+    MakeErrMsg(ErrMsg, "OpenProcess failed!");
+    return -1;
+  }
+  AutoHandle hProcess(hOpen);
 
   // Wait for the process to terminate.
   DWORD millisecondsToWait = INFINITE;
@@ -341,12 +327,18 @@ Program::Wait(unsigned secondsToWait,
 
 bool
 Program::Kill(std::string* ErrMsg) {
-  if (Data_ == 0) {
+  if (Pid_ == 0) {
     MakeErrMsg(ErrMsg, "Process not started!");
     return true;
   }
 
-  HANDLE hProcess = reinterpret_cast<HANDLE>(Data_);
+  HANDLE hOpen = OpenProcess(PROCESS_TERMINATE, FALSE, Pid_);
+  if (hOpen == NULL) {
+    MakeErrMsg(ErrMsg, "OpenProcess failed!");
+    return true;
+  }
+  AutoHandle hProcess(hOpen);
+
   if (TerminateProcess(hProcess, 1) == 0) {
     MakeErrMsg(ErrMsg, "The process couldn't be killed!");
     return true;