[Lldb-commits] [PATCH] D111964: [lldb] [lldb-server] Allow any protocol supported by lldb

[Michał Górny via Phabricator via lldb-commits]

mgorny added a comment.

In D111964#3069414 <https://reviews.llvm.org/D111964#3069414>, @labath wrote:

> What's the test strategy for this?

Still thinking about it. I'm mostly PoC-ing this while other changes are waiting.

>> This is not fully functional yet, as lldb-server
>> crashes when attempting to send long packets (e.g. target.xml contents).
>
> I am guessing you'll also want to disable QStartNoAckMode for these connections (?)

I suppose this makes sense. Are you suggesting special-casing `serial://` or do you have something else in mind?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111964/new/

https://reviews.llvm.org/D111964