WebCore/bridge/qt/qt_runtime.cpp:34: Alphabetical sorting problem. [build/include_order] [4] WebCore/bridge/qt/qt_runtime.cpp:68: Should have a space between // and comment [whitespace/comments] [4] WebCore/bridge/qt/qt_runtime.cpp:69: Should have a space between // and comment [whitespace/comments] [4] WebCore/bridge/qt/qt_runtime.cpp:72: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:74: Missing space before { [whitespace/braces] [5] WebCore/bridge/qt/qt_runtime.cpp:75: Missing space before { [whitespace/braces] [5] WebCore/bridge/qt/qt_runtime.cpp:137: 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_runtime.cpp:148: Declaration has space between type name and * in JSObject *object [whitespace/declaration] [3] WebCore/bridge/qt/qt_runtime.cpp:149: One space before end of line comments [whitespace/comments] [5] WebCore/bridge/qt/qt_runtime.cpp:149: 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_runtime.cpp:193: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bridge/qt/qt_runtime.cpp:192: Missing space before ( in switch( [whitespace/parens] [5] WebCore/bridge/qt/qt_runtime.cpp:245: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/bridge/qt/qt_runtime.cpp:320: 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_runtime.cpp:344: Missing space before ( in while( [whitespace/parens] [5] WebCore/bridge/qt/qt_runtime.cpp:639: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:638: 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_runtime.cpp:749: 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_runtime.cpp:791: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:792: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:793: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:794: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:804: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:805: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:806: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:807: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:808: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:809: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:810: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:811: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:812: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:827: 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_runtime.cpp:834: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:835: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:839: Missing space after , [whitespace/comma] [3] WebCore/bridge/qt/qt_runtime.cpp:913: 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_runtime.cpp:945: Missing space after , [whitespace/comma] [3] WebCore/bridge/qt/qt_runtime.cpp:984: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:1021: More than one command on the same line [whitespace/newline] [4] WebCore/bridge/qt/qt_runtime.cpp:1053: 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_runtime.cpp:1061: This { should be at the end of the previous line [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:1078: Missing spaces around = [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:1149: Missing space before ( in foreach( [whitespace/parens] [5] WebCore/bridge/qt/qt_runtime.cpp:1158: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] WebCore/bridge/qt/qt_runtime.cpp:1184: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] WebCore/bridge/qt/qt_runtime.cpp:1187: One line control clauses should not use braces. [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:1246: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] WebCore/bridge/qt/qt_runtime.cpp:1245: 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_runtime.cpp:1254: One line control clauses should not use braces. [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:1273: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] WebCore/bridge/qt/qt_runtime.cpp:1336: Missing spaces around = [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:1392: Declaration has space between type name and * in QObject *obj [whitespace/declaration] [3] WebCore/bridge/qt/qt_runtime.cpp:1395: Declaration has space between type name and * in void *qargs [whitespace/declaration] [3] WebCore/bridge/qt/qt_runtime.cpp:1425: 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_runtime.cpp:1542: 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_runtime.cpp:1552: Declaration has space between type name and * in JSObject *asObj [whitespace/declaration] [3] WebCore/bridge/qt/qt_runtime.cpp:1569: One line control clauses should not use braces. [whitespace/braces] [4] WebCore/bridge/qt/qt_runtime.cpp:1570: 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_runtime.cpp:1577: 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_runtime.cpp:1583: 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_runtime.cpp:1605: An else should appear on the same line as the preceding } [whitespace/newline] [4] WebCore/bridge/qt/qt_runtime.cpp:1598: 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_runtime.cpp:1614: Missing space before ( in foreach( [whitespace/parens] [5] WebCore/bridge/qt/qt_runtime.cpp:1709: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] WebCore/bridge/qt/qt_runtime.cpp:1710: One space before end of line comments [whitespace/comments] [5] WebCore/bridge/qt/qt_runtime.cpp:1711: One space before end of line comments [whitespace/comments] [5] WebCore/bridge/qt/qt_runtime.cpp:1717: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] WebCore/bridge/qt/qt_runtime.cpp:1720: One space before end of line comments [whitespace/comments] [5] WebCore/bridge/qt/qt_runtime.cpp:1737: Declaration has space between type name and * in void *QtConnectionObject [whitespace/declaration] [3] WebCore/bridge/qt/qt_runtime.cpp:1739: More than one command on the same line in if [whitespace/parens] [4] WebCore/bridge/qt/qt_runtime.cpp:1745: _c is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] WebCore/bridge/qt/qt_runtime.cpp:1745: _id is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] WebCore/bridge/qt/qt_runtime.cpp:1780: l is incorrectly named. Don't use the single letter 'l' as an identifier name. [readability/naming] [4] WebCore/bridge/qt/qt_runtime.cpp:1784: Missing spaces around = [whitespace/operators] [4] WebCore/bridge/qt/qt_runtime.cpp:1784: Missing space before ( in for( [whitespace/parens] [5] WebCore/bridge/qt/qt_runtime.cpp:1839: Extra space before ( in function call [whitespace/parens] [4] WebCore/bridge/qt/qt_runtime.cpp:1858: One line control clauses should not use braces. [whitespace/braces] [4] Total errors found: 77 in 1 files
Created attachment 57109 [details] Patch that fixes the coding style issues
*** This bug has been marked as a duplicate of bug 39746 ***