[Lldb-commits] [lldb] [lldb] correct inconsistent order of messages on process launch (PR #73173)

[David Spickett via lldb-commits]

================
@@ -265,8 +265,6 @@ class CommandObjectProcessLaunch : public 
CommandObjectProcessLaunchOrAttach {
         process_sp->SyncIOHandler(0, std::chrono::seconds(2));
 
         llvm::StringRef data = stream.GetString();
----------------
DavidSpickett wrote:

If we don't need `data` you could move it down to where it's used.

I'd say do `if (llvm::StringRef data = stream.GetString())...` but I don't 
think that will work because Stringref doesn't have `operator bool`.

https://github.com/llvm/llvm-project/pull/73173
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits