[Lldb-commits] [PATCH] D93225: [lldb] Add helper class for dealing with key:value; GDB responses

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Dec 17 12:36:23 PST 2020

labath added a comment.

I think that streamlining the packet (de)serialization process is badly needing. I'm not so sure about the approach though. On one hand, it's definitely an improvement. On the other, it does not handle the deserialization process, and I was hoping we could have a solution which handles both, for two reasons:

- it makes things simpler
- it allows us to avoid the need for tests which explicitly test serialization and deserialization of each packet, as the things is implemented (and tested) once, centrally.

Have you looked at the how llvm YAML and JSON libraries handle (de)serialization? I was hoping that we could implement something similar to that...

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list