[Lldb-commits] [lldb] [lldb-dap] Change the launch sequence (PR #138219)

[Jonas Devlieghere via lldb-commits]

================
@@ -417,8 +424,11 @@ struct DAP {
   lldb::SBMutex GetAPIMutex() const { return target.GetAPIMutex(); }
 
 private:
-  std::mutex m_queue_mutex;
+  /// Queue for all incoming messages.
   std::deque<protocol::Message> m_queue;
+  /// Dedicated queue for launching and attaching.
+  std::deque<protocol::Message> m_launch_attach_queue;
----------------
JDevlieghere wrote:

I changed it to an optional, but then backed it out again because we also have to delay the initial `threads` request until we've completed the launch or attach. Let me know if you prefer two optionals instead (i.e. `m_pending_launch_or_attach` and `m_pending_threads`).

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