[Lldb-commits] [lldb] [lldb-dap] Add new optional argument `time-to-live` when using `--connection` (PR #156803)

via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 8 09:26:39 PDT 2025


================
@@ -327,6 +366,11 @@ serveConnection(const Socket::SocketProtocol &protocol, const std::string &name,
       std::unique_lock<std::mutex> lock(dap_sessions_mutex);
       dap_sessions.erase(&loop);
       std::notify_all_at_thread_exit(dap_sessions_condition, std::move(lock));
+
+      // Start the countdown to kill the server at the end of each connection.
----------------
jeffreytan81 wrote:

Just realize that we used `loop.AddCallback` to run the callback at the main loop thread not each DAP's worker thread. Then the race condition mentioned above won't happen and no synchronization is required. 

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


More information about the lldb-commits mailing list