[Lldb-commits] [lldb] [lldb][progress] Correctly check total for deterministic progress (PR #79912)

Chelsea Cassanova via lldb-commits lldb-commits at lists.llvm.org
Tue Jan 30 11:26:26 PST 2024


================
@@ -39,7 +39,7 @@ class ProgressEventData : public EventData {
   GetAsStructuredData(const Event *event_ptr);
 
   uint64_t GetID() const { return m_id; }
-  bool IsFinite() const { return m_total != UINT64_MAX; }
+  bool IsFinite() const { return m_total != 1; }
----------------
chelcassanova wrote:

I added your suggestions to the PR, no longer using a magic number or the number 1 to represent non-deterministic progress

https://github.com/llvm/llvm-project/pull/79912


More information about the lldb-commits mailing list