JackyAndroid / AndroidTVLauncher

This is a leanback style tv launcher(minSdkVersion 17)
GNU General Public License v3.0
3.08k stars 739 forks source link

Multiple quality improvements #4

Closed m-ezzat closed 8 years ago

m-ezzat commented 8 years ago

This pull request is focused on resolving occurrences of Sonar rules squid:S1170 - Public constants and fields initialized at declaration should be "static final" rather than merely "final" squid:S00117 - Local variable and method parameter names should comply with a naming convention squid:S1488 - Local Variables should not be declared and then immediately returned or thrown squid:S2293 - The diamond operator ("<>") should be used

You can find more information about the issues here: https://dev.eclipse.org/sonar/coding_rules#q=squid:S1170 https://dev.eclipse.org/sonar/coding_rules#q=squid:S00117 https://dev.eclipse.org/sonar/coding_rules#q=squid:S1488 https://dev.eclipse.org/sonar/coding_rules#q=squid:S2293

Please let me know if you have any questions.

M-Ezzat