djasper added a comment.
That's what I meant by "The name NamespaceIndentation might then be a bit
confusing, but not sure whether it's worth changing it.".
I am honestly not sure. Let's get a third opinion.
If we add this additional option, I think we need to fix the behavior and make
what people expect:
With NI_None:
namespace A { namespace B {
int i;
namespace C {
int j;
} // namespace C
}} // namespace A::B
With NI_Inner:
namespace A { namespace B {
int i;
namespace C {
int j;
} // namespace C
}} // namespace A::B
With NI_All:
namespace A { namespace B {
int i;
namespace C {
int j;
} // namespace C
}} // namespace A::B
https://reviews.llvm.org/D32480
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
