================
@@ -494,6 +494,49 @@ void ThreadList::DiscardThreadPlans() {
     (*pos)->DiscardThreadPlans(true);
 }
 
+void ThreadList::SetUpStepOverBreakpointBeforeResumeIfNeeded(
----------------
jimingham wrote:

That's better.  It still doesn't stop someone from adding so code below this 
that isn't breakpoint related, which will then work on every platform but yours 
because they didn't notice this return.  If the early return & breakpoint 
handling were in a separate method, then that mistake wouldn't be possible.
Not terribly important, however.

https://github.com/llvm/llvm-project/pull/165760
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to