[PATCH] D46274: [Support] Harden JSON against invalid UTF-8.
Sam McCall via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 30 11:22:50 PDT 2018
sammccall added a subscriber: benhamilton.
sammccall added a comment.
Thanks for the comments!
Not validating codepoints was deliberate but in hindsight arbitrary, I'll
fix that. (Both surrogates and high codepoints).
Regarding a separate library: I agree in principle :-) but am concerned
about the potential scope/generality and don't have cycles to design it
right now, even extracting JSON is a bit of a yak-shave...
Here it's in support and exposed as free functions, so if it really is
useful, it *is* available and we can move it later.
That's the best argument I can come up with, open to better ideas :)
Repository:
rL LLVM
https://reviews.llvm.org/D46274
More information about the llvm-commits
mailing list