mib marked 3 inline comments as done.
mib added inline comments.
================
Comment at: lldb/source/Plugins/Platform/scripted/ScriptedPlatform.cpp:261
+ if (!proc_info_or_error) {
+ llvm::consumeError(proc_info_or_error.takeError());
+ return false;
----------------
labath wrote:
> When I saw the amount of effort put into the error messages in the
> ParseProcessFunction, I assumed you are going to give those messages to the
> user somehow, but now I see they are just thrown away. Do you plan to change
> that? Maybe you could at least log (LLDB_LOG_ERROR) them?
I wish there was a way to surface these error but the current implementation of
the platform plugin doesn't allow it. I guess I'll just log them for now.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139252/new/
https://reviews.llvm.org/D139252
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
