[Lldb-commits] [lldb] [lldb-dap] Add new optional argument `time-to-live` when using `--connection` (PR #156803)
Roy Shi via lldb-commits
lldb-commits at lists.llvm.org
Mon Sep 8 11:46:34 PDT 2025
royitaqi wrote:
@JDevlieghere, @walter-erquinigo: I have updated the patch according to your suggestions. LMK if it looks alright.
Changes:
1. Rename the option to `--connection-timeout`.
2. Print a warning and exit lldb-dap when `--connection-timeout` is given and `--connection` isn't.
3. Add an extension setting (similar to `Server Mode`) and pass that down to `--connection-timeout` when starting the server.
4. Add api tests.
Caveats:
* The existing tests in `TestDAP_server.py` don't test with concurrent clients. My newly added tests don't either.
* Wanted to update `lldb/tools/lldb-dap/README.md`, but realize that the "Server Mode" isn't mentioned there. @walter-erquinigo LMK if you want me to add a section to mention both (if so, where in the doc and suggestive new section name?).
https://github.com/llvm/llvm-project/pull/156803
More information about the lldb-commits
mailing list