[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 mailing list