[Lldb-commits] [PATCH] D128768: [lldb/Core] Fix finite progress event reporting

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jun 29 13:55:30 PDT 2022


JDevlieghere added inline comments.


================
Comment at: lldb/source/Core/Debugger.cpp:1839-1855
+    if (log && log->GetVerbose()) {
+      if (data->IsFinite())
+        LLDB_LOGF(log,
+                  "%p Debugger('%llu')::HandleProgressEvent "
+                  "(m_current_event_id = {%llu}, data = {id = %llu, message = "
+                  "%s, completed = %llu, total = %llu})",
+                  static_cast<void *>(this), m_uid, *m_current_event_id, id,
----------------
What's `m_uid`?

I would either fold these two log statements into one, and use `LLDB_LOGV(GetLog(LLDBLog::Events), ...` or I would use the event's dump method:

```
if (log && log->GetVerbose()) {
  StreamString log_stream;
  log_stream << "Debugger::HandleProgressEvent" ... 
  data->dump(log_stream);
  log->PutString(log_stream.GetString());
}
```


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128768/new/

https://reviews.llvm.org/D128768



More information about the lldb-commits mailing list