atirit added a comment.
OK, I'm getting a few more failed unit tests and I'm really not sure what
correct formatting behaviour is anymore, so I'll just ask.
Assuming the following settings:
BreakBeforeBraces: Custom
BraceWrapping: {
AfterEnum: true
}
AllowShortEnumsOnASingleLine: true
Which of these is correct? (`FormatTest,AllmanBraceBreaking`)
A (expected in test):
enum X
{
Y = 0
}
B:
enum X { Y = 0 }
Which of these is correct? (new test)
A:
enum
{
A,
B
} Enum1, Enum2
B:
enum
{
A,
B
} Enum1,
Enum2
C:
enum { A, B } Enum1, Enum2
`true/true` results in option B of the "new test" section. The strange
behaviour with `Enum1` and `Enum2` occurs when `AllowShortEnumsOnASingleLine`
isn't allowed to shrink a line for whatever reason, be it a comma on the last
case, a comment, or too long an `enum`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93938/new/
https://reviews.llvm.org/D93938
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
