diff options
| author | Jim Ingham <jingham@apple.com> | 2013-07-18 21:48:26 +0000 |
|---|---|---|
| committer | Jim Ingham <jingham@apple.com> | 2013-07-18 21:48:26 +0000 |
| commit | 008f3dc79c762b7d240b5ad6d4fb148c5cb039ca (patch) | |
| tree | c4820585f4f93197df36c77c20b4f3a6371a6fd3 /include/lldb/Target/ThreadPlanBase.h | |
| parent | 3138ad6178b98337ea875e30e7c808e35eae734d (diff) | |
This commit does two things. One, it converts the return value of the QueueThreadPlanXXX
plan providers from a "ThreadPlan *" to a "lldb::ThreadPlanSP". That was needed to fix
a bug where the ThreadPlanStepInRange wasn't checking with its sub-plans to make sure they
succeed before trying to proceed further. If the sub-plan failed and as a result didn't make
any progress, you could end up retrying the same failing algorithm in an infinite loop.
<rdar://problem/14043602>
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@186618 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/lldb/Target/ThreadPlanBase.h')
| -rw-r--r-- | include/lldb/Target/ThreadPlanBase.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/lldb/Target/ThreadPlanBase.h b/include/lldb/Target/ThreadPlanBase.h index 226caeab8..69959e12f 100644 --- a/include/lldb/Target/ThreadPlanBase.h +++ b/include/lldb/Target/ThreadPlanBase.h @@ -59,7 +59,7 @@ protected: ThreadPlanBase (Thread &thread); private: - friend ThreadPlan * + friend lldb::ThreadPlanSP Thread::QueueFundamentalPlan(bool abort_other_plans); DISALLOW_COPY_AND_ASSIGN (ThreadPlanBase); |