[PATCH] D31992: [clangd] Escape only necessary characters in JSON output
Marc-Andre Laperle via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 13 07:11:09 PDT 2017
malaperle-ericsson added a comment.
In https://reviews.llvm.org/D31992#725913, @joerg wrote:
> I'm strongly against this patch. Can you give an actual test case for the problematic behavior?
Sure I can add a test. If you meant more real work scenario, you can juste type "é" in VS Code and it will throw an exception trying to parse \x
Repository:
rL LLVM
https://reviews.llvm.org/D31992
More information about the cfe-commits
mailing list