[PATCH] D32480: clang-format: Add CompactNamespaces option

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 23 05:32:12 PDT 2017


Typz added a comment.

In https://reviews.llvm.org/D32480#761862, @krasimir wrote:

> In any case, adding a namespace end comment to a line closing multiple namespaces is super confusing for me: what does the comment refer to: the inner one, the outer one, or both?
>  `}} // namespace A::B`


it refers to both, similar to C++17' _nested namespace_ definition:

  namespace A::B {
  }} // namespace A::B


https://reviews.llvm.org/D32480





More information about the cfe-commits mailing list