Re: [PATCH v3 05/14] mingw: use real pid
- From
Erik Faye-Lund <kusmabite@gmail.com>
- Date
- Oct 10, 2010, 20:52 UTC
- Message-ID
- <AANLkTinmpzo2-eyPrnx0u=tGBOcMBz03LuFUjZO87dG5@mail.gmail.com>
- In-Reply-To
- <4CB219D3.8000801@sunshineco.com>
On Sun, Oct 10, 2010 at 9:53 PM, Eric Sunshine <sunshine@sunshineco.com> wrote:
Show 10 quoted lines
> On 10/10/2010 9:20 AM, Erik Faye-Lund wrote: >> >> The Windows port so far used process handles as PID. However, >> this does not work consistently with getpid. > > Perhaps this could be elaborated a bit to explain the interaction with > getpid() and how it is causing problems for daemon mode. For the casual > reader, it is not immediately obvious what is failing or why this patch is > needed. >
Good point. How about something like this?
"The Windows port so far used process handles as PID. However, this is not consistent with what getpid returns.
PIDs are system-global identifiers, but process handles are local to a process. Using PIDs instead of process handles allows for instance a user to kill a hung process with the Task Manager, something that would have been impossible with process handles."