clayborg added inline comments.

================
Comment at: lldb/tools/lldb-vscode/lldb-vscode.cpp:796
+    if (!error.Success())
+      response.try_emplace("error", error.GetCString());
     g_vsc.debugger.SetAsync(true);
----------------
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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81200



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to