[Lldb-commits] [PATCH] D112046: [lldb/bindings] Change ScriptedThread initializer parameters

Med Ismail Bennani via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 22 13:31:58 PDT 2021


mib added a comment.

In D112046#3081556 <https://reviews.llvm.org/D112046#3081556>, @mib wrote:

> In D112046#3081547 <https://reviews.llvm.org/D112046#3081547>, @JDevlieghere wrote:
>
>> Given that there's a 1:1 mapping between the target and the process, why did you change the interface as opposed to say extracting the process from the target?
>
> A process is guaranteed to have a target, but the inverse is not true: if the process didn't launch or failed to launch, the target has no process.

To add more to this, I can't think of any reason, at the moment, to create a ScriptedThread if you don't have ScriptedProcess.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112046



More information about the lldb-commits mailing list