https://bugs.llvm.org/show_bug.cgi?id=40840

            Bug ID: 40840
           Summary: clang-format violates WebKit: braces empty block
           Product: clang
           Version: 7.0
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: Formatter
          Assignee: unassignedclangb...@nondot.org
          Reporter: rodionmal...@gmail.com
                CC: djas...@google.com, kli...@google.com,
                    llvm-bugs@lists.llvm.org

Formatting using webkit style violates style guide
https://webkit.org/code-style-guidelines/#braces-empty-block

config:
Language: Cpp
BasedOnStyle: WebKit

1) Case with empty block
having 
for (int i = 0; i < 0; ++i, doBar()) { }
after formatting I've got
for (int i = 0; i < 0; ++i, doBar()) {
} 

2) Case with empty function body. Strictly speaking I wasn't able to find exact
requirement for this but I've got warning from linter (check-webkit-style)

having
void Foo() { }
after formating I've got
void Foo() {}

warning is
Missing space inside { }.  [whitespace/braces] [5]

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to