[Lldb-commits] [PATCH] D41069: NPL: Clean up handling of inferior exit

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 11 09:18:22 PST 2017


clayborg added a comment.

Is the lldb_private::Process we have an exit status in the class itself. The first person to set the exit status wins and no one can set it twice. Doesn't look like what we are doing here. I am not able to tell what actually fixes things here?



================
Comment at: unittests/tools/lldb-server/tests/TestClient.cpp:43
 TestClient::~TestClient() {
+  if( !IsConnected())
+    return;
----------------
space after if and not after (


https://reviews.llvm.org/D41069





More information about the lldb-commits mailing list