[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