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

John Harrison via lldb-commits lldb-commits at lists.llvm.org
Thu May 1 17:13:29 PDT 2025


================
@@ -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;
----------------
ashgti wrote:

I think there is only ever one launch or attach, should this be an optional message instead? Also named `m_pending_launch_or_attach`?

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


More information about the lldb-commits mailing list