[lldb] Avoid unnecessary statusline redraw in HandleProgressEvent

JDevlieghere · JDevlieghere · commit 0e52fa290050 · 2025-03-28T21:51:40.000-07:00
There's no need to call RedrawStatusline from HandleProgressEvent. The statusline gets redraw after handling all events, including progress events, in the default event handler loop. (cherry picked from commit 618e8b9)
diff --git a/lldb/source/Core/Debugger.cpp b/lldb/source/Core/Debugger.cpp
@@ -2212,8 +2212,6 @@ void Debugger::HandleProgressEvent(const lldb::EventSP &event_sp) {
       m_progress_reports.push_back(progress_report);
     }
   }
-
-  RedrawStatusline();
 }
 
 std::optional<Debugger::ProgressReport>

Original file line number	Diff line number	Diff line change
`@@ -2212,8 +2212,6 @@ void Debugger::HandleProgressEvent(const lldb::EventSP &event_sp) {`
`2212`	`2212`	`m_progress_reports.push_back(progress_report);`
`2213`	`2213`	`}`
`2214`	`2214`	`}`
`2215`		`-`
`2216`		`- RedrawStatusline();`
`2217`	`2215`	`}`
`2218`	`2216`
`2219`	`2217`	`std::optional<Debugger::ProgressReport>`