[Lldb-commits] [PATCH] D33778: Add a NativeProcessProtocol Factory class

Kamil Rytarowski via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Jul 3 09:00:44 PDT 2017

krytarowski added inline comments.

Comment at: source/Plugins/Process/NetBSD/NativeProcessNetBSD.cpp:85
+  int wstatus;
+  ::pid_t wpid = llvm::sys::RetryAfterSignal(-1, ::waitpid, pid, &wstatus, 0);
+  assert(wpid == pid);
labath wrote:
> krytarowski wrote:
> > We can set BSD specific: `-1` -> `WAIT_ANY`.
> > 
> > I verified that it exists on NetBSD, FreeBSD and OpenBSD.
> I think that's semantically incorrect, WAIT_ANY means "any process" when used in the pid argument of waitpid. The -1 here is the "failure value" returned by waitpid, and that one is documented as -1 even on netbsd.
Right, my mistake.


More information about the lldb-commits mailing list