https://github.com/JDevlieghere requested changes to this pull request.

I think this is going in the right direction but I think the interaction 
between the ProgressManager and the Debugger needs to change:

At a high level I'm expecting:

1. Progress object is created
2. Progress is added to the ProgressManager
3. Debugger::ReportProgress is called with (`eBroadcastBitProgress | 
eBroadcastBitProgressCategory`)
4. Incremental updates call directly into Debugger::ReportProgress without 
going through the manager, with just `Debugger::ReportProgress`.
5. When progress is complete, we go through the progress manager again and 
report the final progress to both broadcast bits: `eBroadcastBitProgress | 
eBroadcastBitProgressCategory`


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

Reply via email to