djasper added a comment.
Got two responses so far.
1. Having to closing braces in the same column is weird, but not that weird.
Consider
namespace n {
void f() {
...
}
}
2. Richard Smith (code owner of Clang) says that he doesn't really like the two
closing braces in the same column, but he always cheats by removing the braces
for the last case label / default. You never need them. In any case he'd
strongly prefer the current behavior over adding an extra break and more
indentation.
In conclusion, I don't think we want to move forward with making clang-format do
switch (x) {
case 1:
{
doSomething();
}
}
even with IndentCaseLabels: false.
Repository:
rC Clang
https://reviews.llvm.org/D43183
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
