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
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
