[PATCH] D118869: [clang-format] Non-latin comment prefix whitespace

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 4 06:10:43 PST 2022


krasimir added a comment.

Thank you!
Turns out the original source wasn't using an ascii space for the comment indentation but a unicode [c2 a0] no-break space:

  2f 2f c2 a0 43 6f 6d 6d  65 6e 74 0a 69 6e 74 20  |//..Comment.int |

IMO we shouldn't aim to handle non-ascii spaces for indentation in clang-format; the new behavior on this example is fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118869



More information about the cfe-commits mailing list