[PATCH] D30688: [clang-format] Support namespaces ending in semicolon

2017-03-07 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL297140: [clang-format] Support namespaces ending in semicolon (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D30688?vs=90839=90841#toc Repository: rL LLVM

[PATCH] D30688: [clang-format] Support namespaces ending in semicolon

2017-03-07 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 90839. krasimir added a comment. - Remove braces https://reviews.llvm.org/D30688 Files: lib/Format/NamespaceEndCommentsFixer.cpp unittests/Format/NamespaceEndCommentsFixerTest.cpp Index: unittests/Format/NamespaceEndCommentsFixerTest.cpp

[PATCH] D30688: [clang-format] Support namespaces ending in semicolon

2017-03-07 Thread Daniel Jasper via Phabricator via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Minor nit, otherwise looks good. Comment at: lib/Format/NamespaceEndCommentsFixer.cpp:152 +const FormatToken *EndCommentNextTok = EndCommentPrevTok->Next; +if