diff options
author | Jim Ingham <jingham@apple.com> | 2010-06-19 04:45:32 +0000 |
---|---|---|
committer | Jim Ingham <jingham@apple.com> | 2010-06-19 04:45:32 +0000 |
commit | b01e742af7651361c2e49d22140dee9342d8d72e (patch) | |
tree | ed6662bee21de01cbe59c76e620f2881ac921c1b /lldb/source/Target/Process.cpp | |
parent | b2a38a7286d20a4c6cf20820a0cfe9a1d30df93e (diff) | |
download | bcm5719-llvm-b01e742af7651361c2e49d22140dee9342d8d72e.tar.gz bcm5719-llvm-b01e742af7651361c2e49d22140dee9342d8d72e.zip |
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
Diffstat (limited to 'lldb/source/Target/Process.cpp')
-rw-r--r-- | lldb/source/Target/Process.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lldb/source/Target/Process.cpp b/lldb/source/Target/Process.cpp index 9a6ab0c83e8..970e1e48994 100644 --- a/lldb/source/Target/Process.cpp +++ b/lldb/source/Target/Process.cpp @@ -1313,7 +1313,7 @@ Process::ShouldBroadcastEvent (Event *event_ptr) // We've stopped. First see if we're going to restart the target. // If we are going to stop, then we always broadcast the event. // If we aren't going to stop, let the thread plans decide if we're going to report this event. - // If no thread has an opinion, we also report it. + // If no thread has an opinion, we don't report it. if (state != eStateInvalid) { @@ -1326,8 +1326,6 @@ Process::ShouldBroadcastEvent (Event *event_ptr) case eVoteYes: Process::ProcessEventData::SetRestartedInEvent (event_ptr, true); case eVoteNoOpinion: - return_value = true; - break; case eVoteNo: return_value = false; break; |