[PATCH] D37813: clang-format: better handle namespace macros
Francois Ferrand via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu May 23 02:52:57 PDT 2019
Typz added a comment.
> Ah, wow, I see what confused me now:
> In the fixNamespaceEndComment tests you use an indent that clang-format would not produce. Can you please fix that, otherwise I find this super confusing :)
Can you point a specific exemple (in code) ?
There are many tests related to fixNamespaceEndComment(), which look fine to me and which in many cases are copied from similarly indented test cases with standard namespace's...
(Or was it an earlier problem, and I should fix all these existing test cases as well?)
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D37813/new/
https://reviews.llvm.org/D37813
More information about the cfe-commits
mailing list