[Webkit-unassigned] [Bug 27378] New: cpplint has some issues with one lines if-sentences

[bugzilla-daemon at webkit.org]

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

           Summary: cpplint has some issues with one lines if-sentences
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: kenneth.christiansen at openbossa.org


The following is legal according to the coding style document, but cpplint
complains about it.

At least I was told that on the IRC once.

if (something) {
    method1();
    method2();
} else {
    method3();
}

Example:

WebKit/qt/Api/qwebpage.cpp:2409:  One line control clauses should not use
braces.  [whitespace/braces] [4]

2404         if (subString.isEmpty()) {
2405             d->page->unmarkAllTextMatches();
2406             return true;
2407         } else {
2408             return d->page->markAllMatchesForText(subString,
caseSensitivity, true, 0);
2409         }

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