[PATCH] D31992: [clangd] Escape only necessary characters in JSON output
Marc-Andre Laperle via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Apr 16 12:37:49 PDT 2017
malaperle-ericsson added a comment.
In https://reviews.llvm.org/D31992#726447, @joerg wrote:
> let's escape ... all the known ASCII control characters,
Do you mean encode all of them with \u or keep the two characters representation for those that exist? I think \n is nicer than \u000A and is probably common enough to keep the short version. Same for \r, \t.
More information about the cfe-commits