[Lldb-commits] [lldb] [lldb] Implement coalescing of disjoint progress events (PR #84854)

[Jonas Devlieghere via lldb-commits]

================
@@ -75,45 +81,86 @@ void Progress::ReportProgress() {
   }
 }
 
-ProgressManager::ProgressManager() : m_progress_category_map() {}
+ProgressManager::ProgressManager()
+    : m_alarm(std::chrono::milliseconds(100)), m_entries() {}
----------------
JDevlieghere wrote:

I initially wanted to make this a setting, but you need this when initializing the ProgressManager, which happens way before we even have a debugger or a command interpreter, so there wouldn't be a way to set the setting even. I could move the timeout out of the constructor and allow you to set the timeout after the fact, but that would still leave the question of where to get the setting from: the ProgressManager is shared across debuggers.

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