[Lldb-commits] [PATCH] D97739: Add a progress class that can track and report long running operations that happen in LLDB.
walter erquinigo via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Mar 4 13:43:56 PST 2021
wallace added a comment.
This looks really nice. The only thought I have is that you should accept multiple callbacks instead of only one, as the current implementation discards any existing callback if SetProgressCallback is called twice. I know that it'd be rare that two or more callbacks are registered simultaneously, but it might happen in the future and you could save someone's time by making this a little bit more flexible.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97739/new/
https://reviews.llvm.org/D97739
More information about the lldb-commits
mailing list