[PATCH] D31992: [clangd] Escape only necessary characters in JSON output

Joerg Sonnenberger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 16 13:24:42 PDT 2017


joerg added a comment.

We already have a couple of case that expect the encoding to be compatible. I'm not very attached to the additional special cases from YAML, but having either a common escape function OR a JSON escape in LLVM/Support is quite important.


https://reviews.llvm.org/D31992





More information about the cfe-commits mailing list