[Lldb-commits] [lldb] Make sure that a `Progress` "completed" update is always reported at destruction (PR #102097)
via lldb-commits
lldb-commits at lists.llvm.org
Tue Aug 6 09:51:07 PDT 2024
royitaqi wrote:
> LGTM. Could we cover this scenario by the existing unit test?
Yes, definitely.
At the time of the patch I didn't find any existing unit test files by searching for "ProgressTest.cpp". Just now I realize there is the [ProgressReportTest.cpp](https://github.com/llvm/llvm-project/blob/main/lldb/unittests/Core/ProgressReportTest.cpp). I will read it and try to add a test for this scenario.
https://github.com/llvm/llvm-project/pull/102097
More information about the lldb-commits
mailing list