Cross-platform Java 3D SDK
84
stars
26
forks
source link
multiple code improvements: squid:S1213, squid:S2131, squid:S1155, squid:CommentedOutCodeLine, squid:S00122, squid:S2184, squid:S00115, squid:UselessParenthesesCheck, squid:S1192, squid:S1905 #73
Closed
georgekankava closed 8 years ago
This pull request is focused on resolving occurrences of Sonar rules squid:S1213 - The members of an interface declaration or class should appear in a pre-defined order. squid:S2131 - Primitives should not be boxed just for "String" conversion. squid:S1155 - Collection.isEmpty() should be used to test for emptiness. squid:CommentedOutCodeLine - Sections of code should not be "commented out". squid:S00122 - Statements should be on separate lines. squid:S2184 - Math operands should be cast before assignment. squid:S00115 - Constant names should comply with a naming convention. squid:UselessParenthesesCheck - Useless parentheses around expressions should be removed to prevent any misunderstanding. squid:S1192 - String literals should not be duplicated. squid:S1905 - Redundant casts should not be used. You can find more information about the issue here: https://dev.eclipse.org/sonar/rules/show/squid:S1213 https://dev.eclipse.org/sonar/rules/show/squid:S2131 https://dev.eclipse.org/sonar/rules/show/squid:S1155 https://dev.eclipse.org/sonar/rules/show/squid:CommentedOutCodeLine https://dev.eclipse.org/sonar/rules/show/squid:S00122 https://dev.eclipse.org/sonar/rules/show/squid:S2184 https://dev.eclipse.org/sonar/rules/show/squid:S00115 https://dev.eclipse.org/sonar/rules/show/squid:UselessParenthesesCheck https://dev.eclipse.org/sonar/rules/show/squid:S1192 https://dev.eclipse.org/sonar/rules/show/squid:S1905 Please let me know if you have any questions. George Kankava