[Lldb-commits] [lldb] r232926 - Use POSIX pid_t and not lldb::pid_t.
Davide Italiano
davide at freebsd.org
Sun Mar 22 16:43:59 PDT 2015
Author: davide
Date: Sun Mar 22 18:43:58 2015
New Revision: 232926
URL: http://llvm.org/viewvc/llvm-project?rev=232926&view=rev
Log:
Use POSIX pid_t and not lldb::pid_t.
The latter is uint64_t beacuse lldb supports arbitrary pid/platforms
but in this case we're using it as return value for fork() which might
return -1 to the parent in case the syscall fails.
Differential Revision: http://reviews.llvm.org/D8491
Modified:
lldb/trunk/source/Plugins/Process/FreeBSD/ProcessMonitor.cpp
Modified: lldb/trunk/source/Plugins/Process/FreeBSD/ProcessMonitor.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/FreeBSD/ProcessMonitor.cpp?rev=232926&r1=232925&r2=232926&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/FreeBSD/ProcessMonitor.cpp (original)
+++ lldb/trunk/source/Plugins/Process/FreeBSD/ProcessMonitor.cpp Sun Mar 22 18:43:58 2015
@@ -964,7 +964,7 @@ ProcessMonitor::Launch(LaunchArgs *args)
lldb_utility::PseudoTerminal terminal;
const size_t err_len = 1024;
char err_str[err_len];
- lldb::pid_t pid;
+ ::pid_t pid;
// Propagate the environment if one is not supplied.
if (envp == NULL || envp[0] == NULL)
More information about the lldb-commits
mailing list