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

Reply via email to