[Lldb-commits] [lldb] [lldb-dap] Updating the logging of lldb-dap to use existing LLDBLog. (PR #129294)

John Harrison via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 28 14:46:20 PST 2025


ashgti wrote:

Also, for reference, this does change the output format.

```
$ cat lldb-test-build.noindex/tools/lldb-dap/launch/TestDAP_launch.test_termination/dap.txt
1740771506.578514099 stdin/stdout --> {"command":"disconnect","type":"request","arguments":{},"seq":1}
1740771506.578675032 stdin/stdout <-- {"event":"terminated","seq":0,"type":"event"}
1740771506.579591990 stdin/stdout <-- {"command":"disconnect","request_seq":1,"seq":0,"success":true,"type":"response"}
```

Is an example of what things look like with this change.

Before this was formatted like:

```
<timestamp> <client_name> (<--|-->)
Content-Length: <size>

{...}
```

So its more compact now since we don't include the full protocol encoding header ("Content-Length: {size}\r\n\r\n").

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


More information about the lldb-commits mailing list