JDevlieghere wrote:

> This is only testing the `Progress` class constructor behavior. Could you 
> check that the `Progress::Increment` method works as expected ?

While it would be nice to have full test coverage, the goal was to cover the 
existing behavior which we'll modify/extend to coalesce events as discussed in 
the 
[RFC](https://discourse.llvm.org/t/rfc-improve-lldb-progress-reporting/75717/). 
Increments are orthogonal to that so I think it's fine to keep that for a 
separate PR. 

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

Reply via email to