[Webkit-unassigned] [Bug 30538] New: Replace boolean operator indentation example with an if statement.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 19 14:40:40 PDT 2009


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

           Summary: Replace boolean operator indentation example with an
                    if statement.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit Website
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: scherkus at chromium.org


As Dave Levin pointed out in https://bugs.webkit.org/show_bug.cgi?id=30529 my
|| should have been on the next line.

The style guide gives an example using a "return" statement, however there are
many instances in WebKit today that contradicts that rule and place the
operators at the end of the line.

If "if" statements are not an exception, then I'd suggest replacing the example
to use an if statement.  You're more likely to write if statements than return
statements so this example should prevent further style violations from
happening.

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