[Lldb-commits] [PATCH] D142260: Actually report an error when `command script add` is passed a non-existent class

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

LGTM modulo the else-after-return.



================
Comment at: 
lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp:1949-1950
+        new StructuredPythonObject(std::move(ret_val)));
+  else
+    return {};
 }
----------------
No else after return. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142260

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