[Webkit-unassigned] [Bug 27610] Add check for line-breaking rule #3 to cpplint

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 23 14:42:13 PDT 2009


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


Jakob Petsovits <jpetso at gmx.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33347|0                           |1
        is obsolete|                            |
  Attachment #33379|                            |review?
               Flag|                            |




--- Comment #7 from Jakob Petsovits <jpetso at gmx.at>  2009-07-23 14:42:12 PDT ---
Created an attachment (id=33379)
 --> (https://bugs.webkit.org/attachment.cgi?id=33379)
Add check for line-breaking rule #3 to cpplint (try 2)

New version, incorporating most feedback from Adam Treat and Dave Levin, both
from the comments in here and from comments in IRC.
Now checks for both "else" and "else if" statements, as well as "break" and
"continue" in addition to "return".

This version does not trigger on exit statements in else-if blocks (only in if
blocks) and also does not (yet?) unify regular expressions for goto labels
(which was suggested by Adam). Apart from that, it's pretty neat though. Please
have another look.

-- 
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