[PATCH] D43183: clang-format: introduce `CaseBlockIndent` to control indent in switch
Francois Ferrand via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 13 08:48:30 PST 2018
Typz added a comment.
Hum, not sure I fully got your proposal. So you actually mean that clang-format you format like this (with 4-space indent for clarity):
switch (x) {
case 0:
break;
case 1: {
foo();
break;
}
case 2: {
foo();
} break;
case 3:
{
foo();
}
bar();
break;
}
Is this right?
If so it does not really help me: I don't care so much how it is formatted, but I think the current way is way too error prone (and I cannot change the style to indent the case blocks themself). So i'll have to keep a patch in my fork :-(
Or maybe the behavior should be dependant on `IndentCaseLabels` (though this would change LLVM style formatting) ?
Repository:
rC Clang
https://reviews.llvm.org/D43183
More information about the cfe-commits
mailing list