[Lldb-commits] [PATCH] linux: use IS_VALID_LLDB_HOST_THREAD.

Stephen Wilson wilsons at start.ca
Mon Feb 7 20:59:28 PST 2011


Update the linux plugin code to use the new check for a valid host
thread introduced in r125067.


diff --git a/source/Plugins/Process/Linux/ProcessMonitor.cpp b/source/Plugins/Process/Linux/ProcessMonitor.cpp
index 2c93638..490ca6f 100644
--- a/source/Plugins/Process/Linux/ProcessMonitor.cpp
+++ b/source/Plugins/Process/Linux/ProcessMonitor.cpp
@@ -555,7 +555,7 @@ WAIT_AGAIN:
     // Finally, start monitoring the child process for change in state.
     m_monitor_thread = Host::StartMonitoringChildProcess(
         ProcessMonitor::MonitorCallback, this, GetPID(), true);
-    if (m_monitor_thread == LLDB_INVALID_HOST_THREAD)
+    if (!IS_VALID_LLDB_HOST_THREAD(m_monitor_thread))
     {
         error.SetErrorToGenericError();
         error.SetErrorString("Process launch failed.");
@@ -580,7 +580,7 @@ ProcessMonitor::StartOperationThread(LaunchArgs *args, Error &error)
 {
     static const char *g_thread_name = "lldb.process.linux.operation";
 
-    if (m_operation_thread != LLDB_INVALID_HOST_THREAD)
+    if (IS_VALID_LLDB_HOST_THREAD(m_operation_thread))
         return;
 
     m_operation_thread =
@@ -592,7 +592,7 @@ ProcessMonitor::StopOperationThread()
 {
     lldb::thread_result_t result;
 
-    if (m_operation_thread == LLDB_INVALID_HOST_THREAD)
+    if (!IS_VALID_LLDB_HOST_THREAD(m_operation_thread))
         return;
 
     Host::ThreadCancel(m_operation_thread, NULL);
@@ -1009,7 +1009,7 @@ ProcessMonitor::StopMonitoringChildProcess()
 {
     lldb::thread_result_t thread_result;
 
-    if (m_monitor_thread != LLDB_INVALID_HOST_THREAD)
+    if (IS_VALID_LLDB_HOST_THREAD(m_monitor_thread))
     {
         Host::ThreadCancel(m_monitor_thread, NULL);
         Host::ThreadJoin(m_monitor_thread, &thread_result, NULL);



More information about the lldb-commits mailing list