================
@@ -566,6 +559,53 @@ class ExecutionContext {
   lldb::StackFrameSP m_frame_sp; ///< The stack frame in thread.
 };
 
+/// A wrapper class representing an execution context with non-null Target
+/// and Process pointers, a locked API mutex and a locked ProcessRunLock.
+/// The locks are private by design: to unlock them, destroy the
+/// CompleteExecutionContext.
+struct CompleteExecutionContext : ExecutionContext {
----------------
felipepiovezan wrote:

I had intended to ask for feedback on the name, but forgot to include it in the 
top level message; the suggestion is good IMO, so I'll update the PR.

One other thing that I want to do in a later PR -- in the same one where I 
clean all `if(process && target)` is add "get process" / "get target" methods 
that return references. Sadly I can't shadow the base class methods because 
they are called "GetProcessPtr/GetTargetPtr", so it would be awkward.

https://github.com/llvm/llvm-project/pull/152020
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to