================
@@ -77,16 +77,19 @@ ProgressEvent::Create(uint64_t progress_id,
std::optional<StringRef> message,
if (event.GetEventType() == progressStart && event.GetEventName().empty())
return std::nullopt;
- if (prev_event && prev_event->EqualsForIDE(event))
+ if (prev_event && prev_event->EqualsForIDE(event, total))
return std::nullopt;
return event;
}
-bool ProgressEvent::EqualsForIDE(const ProgressEvent &other) const {
+bool ProgressEvent::EqualsForIDE(const ProgressEvent &other, uint64_t total)
const {
return m_progress_id == other.m_progress_id &&
- m_event_type == other.m_event_type &&
- m_percentage == other.m_percentage;
+ m_event_type == other.m_event_type &&
+ // If we check the percentage of a non-deterministic event
----------------
ashgti wrote:
I think that should be fine though, right? Unless the message or progress %
changes then we shouldn't need to send an update (see
https://microsoft.github.io/debug-adapter-protocol/specification#Events_ProgressUpdate).
If this is to update the message, should we add `m_message` to the Equals
check?
https://github.com/llvm/llvm-project/pull/140162
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
