[Webkit-unassigned] [Bug 27498] Feature request: cpplint should check for braces - rule 3

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 21 15:33:43 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=27498





--- Comment #1 from Kenneth Rohde Christiansen <kenneth.christiansen at openbossa.org>  2009-07-21 15:33:42 PDT ---
Actually I believe that we still have no code checking for "missing" braces.
And this is basically not Qt specific, but a case of a single statement
spanning multiply lines.

Actually I just did the test:

3096         self.assert_multi_line_lint(
3097             'if (true)\n'
3098             '    myFunction(reallyLongParam1, reallyLongParam2,\n'
3099             '               reallyLongParam3);\n'
3100             '\n',
3101             '')

and it didn't catch the error, this should be using { }.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list