[Lldb-commits] [lldb] r179326 - Fixed the thread list so it correctly updates after the first core thread exists.

Greg Clayton gclayton at apple.com
Thu Apr 11 15:23:34 PDT 2013


Author: gclayton
Date: Thu Apr 11 17:23:34 2013
New Revision: 179326

URL: http://llvm.org/viewvc/llvm-project?rev=179326&view=rev
Log:
Fixed the thread list so it correctly updates after the first core thread exists.


Modified:
    lldb/trunk/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp

Modified: lldb/trunk/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp?rev=179326&r1=179325&r2=179326&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp (original)
+++ lldb/trunk/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp Thu Apr 11 17:23:34 2013
@@ -453,10 +453,8 @@ ProcessKDP::GetKernelThread(ThreadList &
     const lldb::tid_t kernel_tid = 1;
     ThreadSP thread_sp (old_thread_list.FindThreadByID (kernel_tid, false));
     if (!thread_sp)
-    {
         thread_sp.reset(new ThreadKDP (*this, kernel_tid));
-        new_thread_list.AddThread(thread_sp);
-    }
+    new_thread_list.AddThread(thread_sp);
     return thread_sp;
 }
 
@@ -471,7 +469,7 @@ ProcessKDP::UpdateThreadList (ThreadList
     if (log && log->GetMask().Test(KDP_LOG_VERBOSE))
         log->Printf ("ProcessKDP::%s (pid = %" PRIu64 ")", __FUNCTION__, GetID());
     
-    // Even though there is a CPU mask, it doesn't mean to can see each CPU
+    // Even though there is a CPU mask, it doesn't mean we can see each CPU
     // indivudually, there is really only one. Lets call this thread 1.
     GetKernelThread (old_thread_list, new_thread_list);
 





More information about the lldb-commits mailing list