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

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 18 09:02:50 PDT 2017


curdeius added a comment.

In https://reviews.llvm.org/D37904#873860, @krasimir wrote:

> Could you please move the test that adds a namespace end comment to `unittests/Format/NamespaceEndCommentsFixerTest.cpp`?


That's what I wanted to do initially, but the very same tests there passed surprisingly. I think that there are some differences related to "messing up" the code between both test suites.
I'll have another look at it however.


https://reviews.llvm.org/D37904





More information about the cfe-commits mailing list