Bug 173859
| Summary: | check-webkit-style complains "multi-line control clauses should use brace" for single line if-statement followed by line with 8 space characters | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | John Wilander <wilander> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | dbates, lforschler |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
John Wilander
This code ...
if (sorted.size() < minimum)
return { };
... results in this checker error:
Multi line control clauses should use braces. [whitespace/braces] [4]
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
John Wilander
I found out why it was complaining. Xcode had indented eight characters, like so:
if (sorted.size() < minimum)
return { };
// Eight space chars here confuses the checker.
Daniel Bates
I hit this bug with attachment #319945 [details] (bug #176388).