[PATCH] D53286: [clangd] Refactor JSON-over-stdin/stdout code into Transport abstraction.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 15 08:20:03 PDT 2018


sammccall added a comment.

For reference, @jkorous has a WIP in https://reviews.llvm.org/D53290.
It's scope is a superset, and I think everything in common is basically the same (they were both based on a common prototype).
Jan is out at the moment, so I think it makes sense to move ahead with this piece - it's progress, and the smaller scope makes landing the change easier.

I just realized though there are no JSONTransport unit tests in this patch, I'll add those.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53286





More information about the cfe-commits mailing list