[Lldb-commits] [PATCH] D129338: Tell the user which pathname was invalid...

David Spickett via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jul 8 01:49:07 PDT 2022


DavidSpickett added inline comments.


================
Comment at: lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp:2640
   if (!pathname || !pathname[0]) {
-    error.SetErrorString("invalid pathname");
+    error.SetErrorString("empty path");
     return false;
----------------
Is this testable? Maybe not because the top level command always wants some string, unless `""` is accepted.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129338



More information about the lldb-commits mailing list