HazardyKnusperkeks added inline comments.
================
Comment at: clang/include/clang/Format/Format.h:469
+ /// Different styles for merging short case labels.
+ enum ShortCaseLabelStyle : int8_t {
+ /// Never merge case code
----------------
MyDeveloperDay wrote:
> HazardyKnusperkeks wrote:
> > While we're at it, shouldn't there be a `Leave`? ;)
> I think Leave is Never
No. `Leave` would leave me with this:
```
switch (a) {
case 1: x = 1; break;
case 2:
return;
}
```
If I'd start with it. `Leave` allows to mix the Yes and No options.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133571/new/
https://reviews.llvm.org/D133571
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
