https://bugs.llvm.org/show_bug.cgi?id=35098
Bug ID: 35098
Summary: clang-format support { on its own line, but only on
multi-line conditionals
Product: clang
Version: unspecified
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P
Component: Formatter
Assignee: unassignedclangb...@nondot.org
Reporter: vladi...@pobox.com
CC: djas...@google.com, kli...@google.com,
llvm-bugs@lists.llvm.org
I'd like to propose an additional brace option for conditionals -- attached
braces if the conditional expression fits on a single line, otherwise a break
before the brace. That is:
if (something < 10) { // one-line condition, attached brace
DoConditionalThing();
}
if (something < 10 && SomeOtherCondition() &&
SomeOtherCheckCall())
{ // multi-line condition, brace on its own
line
DoConditionalThing();
}
Right now, the only options are either putting the { on its own line always
(making the simple statement above 4 lines instead of 3), or attaching it to
the end always, making the complex statement above have a hidden, hanging {,
like so:
if (something < 10 && SomeOtherCondition() &&
SomeOtherCheckCall()) {
DoConditionalThing();
}
I'm willing to take a stab at this myself if there's interest in having this as
an option in BraceWrapping (probably as a "MultilineOnly" option to
BraceWrapping - AfterControlStatement, in addition to true/false)
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
