[Lldb-commits] [PATCH] D128480: [lldb] Replace Host::SystemLog with Debugger::Report{Error, Warning}

Med Ismail Bennani via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jun 23 16:57:04 PDT 2022


mib added inline comments.


================
Comment at: lldb/source/Core/Module.cpp:1187
+
+    Debugger::ReportError(std::string(strm.GetString()));
+  }
----------------
It's unrelated to this patch, but it looks like `Debugger::HandleDiagnosticEvent` dumps everything to the error stream without checking if the event is a warning or an error. I'm mentioning that here because if we did that distinction at the event handling level, we could get rid of `strm.PutCString("error: ")`.

Note however, the reason I'm bringing this up, is that the error message prefix is not consistent with `ReportErrorIfModifyDetected`, since it's not prepended by `error: `.

We should at least fix that for this patch before before making it consistent at the event handler level.


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

https://reviews.llvm.org/D128480



More information about the lldb-commits mailing list