[PATCH] Add non-blocking Wait() for launched processes

Reid Kleckner rnk at google.com
Fri Sep 27 15:02:06 PDT 2013



================
Comment at: include/llvm/Support/Program.h:38
@@ +37,3 @@
+#elif defined(LLVM_ON_WIN32)
+  typedef unsigned long ProcessId; // Must match the type of DWORD on Windows.
+  typedef void * HANDLE; // Must match the type of HANDLE on Windows.
----------------
Rafael Ávila de Espíndola wrote:
> maybe a silly question, but why not use DWORD and HANDLE instead of having these comments?
Please don't include all of windows.h in a header like Program.h that may be included widely in cross-platform code.  :)


http://llvm-reviews.chandlerc.com/D1728



More information about the llvm-commits mailing list