[PATCH] D63462: [analyzer] JsonSupport: Escape escapes

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 18 15:00:04 PDT 2019


Charusso added a comment.

In D63462#1549225 <https://reviews.llvm.org/D63462#1549225>, @NoQ wrote:

> I mean, i'm removing backslashes but you're adding more backslashes. Therefore i think we're talking about different issues.


You *have to* remove backslashes, because sometimes we have `\\\\` so two of them packed together. Then when you remove it, it will be only one (`\\` ), which is handled by JSON properly. Sadly it is very strict and cannot predict two should not be a problem.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63462/new/

https://reviews.llvm.org/D63462





More information about the cfe-commits mailing list