[PATCH] D37904: [clang-format] Fix FixNamespaceComments when BraceWrapping AfterNamespace is true.

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 19 08:41:46 PDT 2017


curdeius added a comment.

I confirm what I observed before: when invoking tests in `unittests/Format/NamespaceEndCommentsFixerTest.cpp`, the `const AnnotatedLine *line` parameter in `getNamespaceToken` gets one big line that includes both `namespace` and `{` (something like `namespace\n{\n...`, whereas in tests invoked from `unittests/Format/FormatTests.cpp`, there is a separate line with `namespace\n` and another one with `{\n`.

I haven't looked at what provokes this behavior, but I imagine that there are additional passes in `FormatTests`.


https://reviews.llvm.org/D37904





More information about the cfe-commits mailing list