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]
Created attachment 57103 [details] Patch that fixes the coding style issues
QtTriaged
Comment on attachment 57103 [details] Patch that fixes the coding style issues Clearing flags on attachment: 57103 Committed r60296: <http://trac.webkit.org/changeset/60296>
All reviewed patches have been landed. Closing bug.