[PATCH] D41290: [YAML] Add support for non-printable characters

[Vladimir Plyashkun via Phabricator via llvm-commits]

vladimir.plyashkun added a comment.

Hello @thegameg !
Is is correct that printable UTF-8 characters are also escaped by these changes?
For example, Clang-Tidy output  now uses double-quotes for UTF-8:

  Message:         "parameter '\xffffffffffffffd0\xffffffffffffffbf\xffffffffffffffd0\xffffffffffffffb0\xffffffffffffffd1\xffffffffffffff80\xffffffffffffffd0\xffffffffffffffb0\xffffffffffffffd0\xffffffffffffffbc\xffffffffffffffd0\xffffffffffffffb5\xffffffffffffffd1\xffffffffffffff82\xffffffffffffffd1\xffffffffffffff80' is unused"

but previously it was simply:

  Message:         'parameter ''параметр'' is unused'

I'm using snakeyaml <https://bitbucket.org/asomov/snakeyaml> which is cannot recognize these sequences. By this i've got incorrect clang-tidy output.
Also i saw that not all the UTF-8 characters are printed in double quotes.
For example in the same output,

  ReplacementText: ' /*параметр*/'

printed in the single quotes.


Repository:
  rL LLVM

https://reviews.llvm.org/D41290