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

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 6 05:05:22 PDT 2017


Typz added a comment.

So how do I proceed?

1. Keep the CompactNamespace option, and make "compacted" namespaces always add at most one level of indentation
2. Or assume that this can only ever usefully work with the behavior of NI_None and add an additional enum value NI_Compact.

And should we limit 'compacting' to namespace which start and end at consecutive lines [e.g. exactly the same conditions as C++17 nested namespaces], or is the current implementation enough [merge all adjacent beginning and all adjacent endings] ?


https://reviews.llvm.org/D32480





More information about the cfe-commits mailing list