Two changes in this checkin. Added a ThreadPlanKind so that I can do some reasoning based on the kind of thread plan

without having to use RTTI.
Removed the ThreadPlanContinue and replaced with a ShouldAutoContinue query that serves the same purpose.  Having to push
another plan to assert that if there's no other indication the target should continue when this plan is popped was flakey
and error prone.  This method is more stable, and fixed problems we were having with thread specific breakpoints.

llvm-svn: 106378

1 files changed, 4 insertions, 4 deletions

diff --git a/lldb/source/Target/ThreadList.cpp b/lldb/source/Target/ThreadList.cpp
index 6bc22712d10..8255c5df3b6 100644
--- a/lldb/source/Target/ThreadList.cpp
+++ b/lldb/source/Target/ThreadList.cpp
@@ -178,7 +178,7 @@ ThreadList::ShouldStop (Event *event_ptr)
     // Running events should never stop, obviously...
 
 
-    bool should_stop = false;
+    bool should_stop = false;    
     m_process->UpdateThreadListIfNeeded();
 
     collection::iterator pos, end = m_threads.end();
@@ -189,12 +189,12 @@ ThreadList::ShouldStop (Event *event_ptr)
     for (pos = m_threads.begin(); pos != end; ++pos)
     {
         ThreadSP thread_sp(*pos);
-        if ((thread_sp->ThreadStoppedForAReason())
-            && (thread_sp->GetResumeState () != eStateSuspended))
+        if ((thread_sp->GetResumeState () != eStateSuspended) && (thread_sp->ThreadStoppedForAReason()))
         {
-            should_stop |= thread_sp->ShouldStop(event_ptr);
+            should_stop |=  thread_sp->ShouldStop(event_ptr);
         }
     }
+
     if (should_stop)
     {
         for (pos = m_threads.begin(); pos != end; ++pos)