[Lldb-commits] [lldb] [lldb][Progress] Separate title and details (PR #77547)
Chelsea Cassanova via lldb-commits
lldb-commits at lists.llvm.org
Wed Jan 10 14:23:04 PST 2024
================
@@ -30,18 +34,19 @@ Progress::~Progress() {
// Make sure to always report progress completed when this object is
// destructed so it indicates the progress dialog/activity should go away.
std::lock_guard<std::mutex> guard(m_mutex);
- if (!m_completed)
- m_completed = m_total;
+ if (m_total.has_value() && !m_completed)
+ m_completed = m_total.value();
ReportProgress();
}
void Progress::Increment(uint64_t amount, std::string update) {
if (amount > 0) {
std::lock_guard<std::mutex> guard(m_mutex);
+ m_details = update;
----------------
chelcassanova wrote:
Good point, I'll add the `std::optional` for `Increment` and remove this parameter from `ReportProgress` 👍🏾
https://github.com/llvm/llvm-project/pull/77547
More information about the lldb-commits
mailing list