[Lldb-commits] [lldb] [lldb][progress][NFC] Add unit test for progress reports (PR #79533)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Fri Jan 26 11:12:57 PST 2024


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


More information about the lldb-commits mailing list