[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