[llvm-bugs] [Bug 35098] New: clang-format support { on its own line, but only on multi-line conditionals

[via llvm-bugs]

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: unassignedclangbugs at nondot.org
          Reporter: vladimir at pobox.com
                CC: djasper at google.com, klimek at google.com,
                    llvm-bugs at 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20171026/f7c123b4/attachment.html>