[Webkit-unassigned] [Bug 39741] New: qt_class.cpp has coding-style errors

[bugzilla-daemon at webkit.org]

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

           Summary: qt_class.cpp has coding-style errors
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit Qt
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: agbakken at gmail.com


WebCore/bridge/qt/qt_class.cpp:21:  Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
WebCore/bridge/qt/qt_class.cpp:23:  Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
WebCore/bridge/qt/qt_class.cpp:28:  Alphabetical sorting problem.  [build/include_order] [4]
WebCore/bridge/qt/qt_class.cpp:144:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WebCore/bridge/qt/qt_class.cpp:148:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
WebCore/bridge/qt/qt_class.cpp:161:  Declaration has space between type name and * in QObject *child  [whitespace/declaration] [3]
WebCore/bridge/qt/qt_class.cpp:198:  Declaration has space between type name and * in QObject *child  [whitespace/declaration] [3]
WebCore/bridge/qt/qt_class.cpp:140:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 8 in 1 files
dfb-47 [flow:~/dev/WebKit/WebCore/bridge/qt]

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