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

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Adam Treat <treat at kde.org>  2009-07-21 15:45:09 PDT ---
Lu(In reply to comment #1)
> 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 { }.

It is a good thing we have a little tool called a *compiler* to tell us that :)

wink wink
nudge nudge

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