[lldb] Update JSONTransport to use MainLoop

[ashgti]

At the moment, JSONTransport is fairly abstract to how its reading/writing.

To support both pipes (e.g. stdin) and sockets, we should update

llvm-project/lldb/include/lldb/Host/JSONTransport.h

Line 69 in d79c284

class JSONTransport {

to perform reads using the RunLoop.

With that, we should also be able to read in larger chunks and use a buffer for tracking partial messages. At the moment, our Read logic ends up reading 1 char at a time to try to find delimiters. This should improve the performance of reading for JSONTransport.

[llvmbot]

@llvm/issue-subscribers-lldb-dap

Author: John Harrison (ashgti)

At the moment, JSONTransport is fairly abstract to how its reading/writing.

To support both pipes (e.g. stdin) and sockets, we should update

llvm-project/lldb/include/lldb/Host/JSONTransport.h

Line 69 in d79c284

class JSONTransport {

to perform reads using the RunLoop.

With that, we should also be able to read in larger chunks and use a buffer for tracking partial messages. At the moment, our Read logic ends up reading 1 char at a time to try to find delimiters. This should improve the performance of reading for JSONTransport.

[llvmbot]

@llvm/issue-subscribers-lldb

Author: John Harrison (ashgti)

At the moment, JSONTransport is fairly abstract to how its reading/writing.

To support both pipes (e.g. stdin) and sockets, we should update

llvm-project/lldb/include/lldb/Host/JSONTransport.h

Line 69 in d79c284

class JSONTransport {

to perform reads using the RunLoop.

With that, we should also be able to read in larger chunks and use a buffer for tracking partial messages. At the moment, our Read logic ends up reading 1 char at a time to try to find delimiters. This should improve the performance of reading for JSONTransport.

[ashgti]

With 1a7b7e2 submitted, we should be able to use MainLoop's on all platforms now.