[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