[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 lldb-commits at lists.llvm.org
Mon Mar 13 16:49:11 PDT 2023


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



More information about the lldb-commits mailing list