[Lldb-commits] [PATCH] D145450: [lldb] Respect empty arguments in target.run-args

[Pavel Labath via Phabricator via lldb-commits]

labath added inline comments.


================
Comment at: lldb/source/Host/common/ProcessLaunchInfo.cpp:324-325
           std::string safe_arg = Args::GetShellSafeArgument(m_shell, argv[i]);
+          if (safe_arg.empty())
+            safe_arg = "\"\"";
           // Add a space to separate this arg from the previous one.
----------------
Shouldn't this belong inside `Args::GetShellSafeArgument`? I don't think there's any situation where the current behavior of the function would be correct (and this is the only caller of the function anyway...)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145450