[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