[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