[PATCH] D33314: clang-format: Add option to remove semicolon at end of namespace

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 19 00:50:45 PDT 2017


djasper added a comment.

I think we should just not do this for now. This addresses a very infrequent case that's easy enough to fix manually. As such, it's not worth the added complexity of clang-format and potential failures it might generate. Changing non-whitespace/non-comment code is always dangerous.

What's the current behavior of the compact namespaces in the presence of semicolons?


https://reviews.llvm.org/D33314





More information about the cfe-commits mailing list