[Lldb-commits] [PATCH] D81200: [vscode] set default values for terminateDebuggee for the disconnect request
walter erquinigo via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jun 11 15:27:01 PDT 2020
wallace marked an inline comment as done.
wallace added inline comments.
Comment at: lldb/tools/lldb-vscode/lldb-vscode.cpp:796
+ if (!error.Success())
+ response.try_emplace("error", error.GetCString());
> I am not sure what returning an error to disconnect will do other than hosing the IDE in some way. It really doesn't matter if we succeeded, we should probably just not mess with the return value. Thoughts?
This is actually quite useful when analyzing logs, as we could know if any disconnection ended up in an internal failure.
This is safe, as I've tried returning a hardcoded error in this case and the IDE terminated the debug session gracefully anyway. The error field is accepted by the protocol, btw.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits