[Lldb-commits] [lldb] [LLDB] Add a target.launch-working-dir setting (PR #113521)

[David Spickett via lldb-commits]

================
@@ -4428,6 +4428,15 @@ void TargetProperties::SetDisableSTDIO(bool b) {
   const uint32_t idx = ePropertyDisableSTDIO;
   SetPropertyAtIndex(idx, b);
 }
+std::optional<llvm::StringRef>
+TargetProperties::GetLaunchWorkingDirectory() const {
----------------
DavidSpickett wrote:

If it's not used when empty should this just return llvm::StringRef?

I get the logical purity of using optional, but if we're defining some value of it as the "null" value anyway we don't have to use optional.

https://github.com/llvm/llvm-project/pull/113521