[Webkit-unassigned] [Bug 27615] Fix false positives for switch statement indentation check in cpplint

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 23 13:18:41 PDT 2009


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


David Levin <levin at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |levin at chromium.org
     Ever Confirmed|0                           |1




--- Comment #2 from David Levin <levin at chromium.org>  2009-07-23 13:18:40 PDT ---
I haven't looked at the patch in depth yet... but it feels like the it should
be not only "return" but also "break" or even "continue". I think the intent is
to not put an "else" if you don't need to.

Ironic, huh ? :)

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