mossberg added a comment.

> So just add a std::string to ThreadPlanStepOut, and cons up your error 
> message there.  Then in ThreadPlanStepOut::ValidatePlan, instead of doing:
> 
>   if (m_return_bp_id == LLDB_INVALID_BREAK_ID) {
>     if (error)
>       error->PutCString("Could not create return address breakpoint.");
>     return false;
>   }
>    
> 
> Write the error string you made in the constructor.
> 
> It's a good idea to log it as well, since the logs don't get command output 
> so having it there will make logs easier to read.

Ok, I see that `ThreadPlanCallFunction.cpp` seems to do this. I'll plan to 
model mine after it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71372/new/

https://reviews.llvm.org/D71372



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to