PMD Results
The following document contains the results of PMD 5.5.1.
Files
com/simpligility/maven/plugins/android/AbstractAndroidMojo.java
Violation |
Line |
Unnecessary use of fully qualified name 'AndroidExtension.APK' due to existing static import 'com.simpligility.maven.plugins.android.common.AndroidExtension.APK' |
92 |
Avoid unused private fields such as 'includeLibsJarsFromApklib'. |
278 |
Avoid unused private fields such as 'includeLibsJarsFromAar'. |
287 |
Avoid unused private fields such as 'artifactHandler'. |
384 |
Avoid unused private fields such as 'envAndroidHome'. |
440 |
Avoid unused private fields such as 'disableConflictingDependenciesWarning'. |
520 |
Useless parentheses. |
671 |
Avoid empty while statements |
840–842 |
Useless parentheses. |
993 |
Useless parentheses. |
993 |
These nested if statements could be combined |
1393–1396 |
com/simpligility/maven/plugins/android/AbstractEmulatorMojo.java
Violation |
Line |
Useless parentheses. |
329 |
Useless parentheses. |
329 |
Useless parentheses. |
377–379 |
Useless parentheses. |
388–390 |
Ternary operators that can be simplified with || or && |
395 |
Avoid empty catch blocks |
405–409 |
Avoid empty catch blocks |
410–414 |
Avoid empty catch blocks |
415–419 |
These nested if statements could be combined |
474–478 |
Useless parentheses. |
494 |
Useless parentheses. |
494 |
Useless parentheses. |
494 |
Do not hard code the IP address |
726 |
Avoid empty catch blocks |
745–748 |
com/simpligility/maven/plugins/android/AbstractInstrumentationMojo.java
Violation |
Line |
Avoid unused private fields such as 'mavenIgnoreTestFailure'. |
67 |
Avoid unused private fields such as 'mavenIgnoreTestError'. |
75 |
com/simpligility/maven/plugins/android/AbstractPublisherMojo.java
Violation |
Line |
Avoid unused private fields such as 'WHATSNEW'. |
28 |
com/simpligility/maven/plugins/android/AndroidSdk.java
Violation |
Line |
These nested if statements could be combined |
473–477 |
com/simpligility/maven/plugins/android/CommandExecutor.java
Violation |
Line |
Avoid modifiers which are implied by the context |
135–138 |
com/simpligility/maven/plugins/android/DefaultJavaProcessExecutor.java
Violation |
Line |
Avoid empty catch blocks |
119–122 |
Avoid empty catch blocks |
129–132 |
Avoid empty catch blocks |
137–140 |
Avoid empty catch blocks |
156–159 |
Avoid empty catch blocks |
166–169 |
Avoid empty catch blocks |
174–177 |
com/simpligility/maven/plugins/android/asm/AnnotatedFinder.java
Violation |
Line |
Avoid unused constructor parameters such as 'parentPackages'. |
22 |
These nested if statements could be combined |
72–75 |
com/simpligility/maven/plugins/android/common/AndroidPublisherHelper.java
Violation |
Line |
Avoid unused private fields such as 'RESOURCES_CLIENT_SECRETS_JSON'. |
66 |
Avoid unused private fields such as 'DATA_STORE_DIR'. |
74 |
Avoid unused private fields such as 'INST_APP_USER_ID'. |
84 |
Avoid unused private methods such as 'checkClientSecretsFile(GoogleClientSecrets)'. |
117 |
com/simpligility/maven/plugins/android/common/DependencyCollector.java
Violation |
Line |
Useless parentheses. |
87 |
com/simpligility/maven/plugins/android/common/DependencyResolver.java
Violation |
Line |
Useless parentheses. |
79 |
com/simpligility/maven/plugins/android/common/JarHelper.java
Violation |
Line |
These nested if statements could be combined |
50–53 |
com/simpligility/maven/plugins/android/common/NativeHelper.java
Violation |
Line |
Avoid empty catch blocks |
247–250 |
com/simpligility/maven/plugins/android/common/PomConfigurationHelper.java
Violation |
Line |
Useless parentheses. |
33 |
These nested if statements could be combined |
51–54 |
com/simpligility/maven/plugins/android/config/ConfigHandler.java
Violation |
Line |
Avoid empty if statements |
87–89 |
Avoid empty catch blocks |
219–222 |
Avoid empty catch blocks |
271–274 |
com/simpligility/maven/plugins/android/configuration/ConfigHelper.java
Violation |
Line |
Useless parentheses. |
46 |
Useless parentheses. |
46 |
Useless parentheses. |
50 |
Avoid empty catch blocks |
67–70 |
Avoid empty catch blocks |
80–83 |
Avoid empty catch blocks |
84–87 |
com/simpligility/maven/plugins/android/configuration/MetaInf.java
Violation |
Line |
Useless parentheses. |
99 |
Useless parentheses. |
100 |
Useless parentheses. |
128 |
com/simpligility/maven/plugins/android/configuration/Monkey.java
Violation |
Line |
Avoid unused private fields such as 'debugNoEvents'. |
71 |
Avoid unused private fields such as 'hprof'. |
75 |
Avoid unused private fields such as 'ignoreCrashes'. |
79 |
com/simpligility/maven/plugins/android/configuration/Proguard.java
Violation |
Line |
Avoid unused private fields such as 'attachMap'. |
33 |
com/simpligility/maven/plugins/android/configuration/Unpack.java
Violation |
Line |
Avoid unused private fields such as 'metaInf'. |
15 |
Avoid unused private fields such as 'lazy'. |
20 |
com/simpligility/maven/plugins/android/phase01generatesources/GenerateSourcesMojo.java
Violation |
Line |
Useless parentheses. |
946 |
Avoid unused private methods such as 'isBuildConfigPresent(Artifact)'. |
1055 |
Useless parentheses. |
1076 |
com/simpligility/maven/plugins/android/phase01generatesources/MavenILogger.java
Violation |
Line |
Useless parentheses. |
33 |
Useless parentheses. |
33 |
Useless parentheses. |
38 |
Useless parentheses. |
38 |
Avoid empty if statements |
39–41 |
com/simpligility/maven/plugins/android/phase04processclasses/ProguardMojo.java
Violation |
Line |
Avoid unused private fields such as 'proguardSkip'. |
94 |
Avoid unused private fields such as 'proguardOptions'. |
109 |
Avoid unused private fields such as 'proguardProguardJarPath'. |
136 |
Avoid unused private fields such as 'outputDirectory'. |
164 |
Avoid unused private fields such as 'proguardJvmArguments'. |
180 |
Avoid unused private fields such as 'proguardFilterMavenDescriptor'. |
189 |
Avoid unused private fields such as 'proguardFilterManifest'. |
198 |
Avoid unused private fields such as 'proguardCustomFilter'. |
209 |
Avoid unused private fields such as 'includeJdkLibs'. |
219 |
Avoid unused private fields such as 'attachMap'. |
228 |
Avoid empty if statements |
667–670 |
Useless parentheses. |
776 |
Useless parentheses. |
776–777 |
com/simpligility/maven/plugins/android/phase08preparepackage/DexMojo.java
Violation |
Line |
Avoid empty if statements |
345–347 |
Avoid empty if statements |
349–352 |
Avoid empty if statements |
354–357 |
com/simpligility/maven/plugins/android/phase09package/AarMojo.java
Violation |
Line |
Avoid unused private fields such as 'ndkClassifier'. |
98 |
Unnecessary use of fully qualified name 'AndroidExtension.AAR' due to existing static import 'com.simpligility.maven.plugins.android.common.AndroidExtension.AAR' |
157 |
com/simpligility/maven/plugins/android/phase09package/ApkMojo.java
Violation |
Line |
Avoid unused private fields such as 'apkNativeToolchain'. |
229 |
Avoid unused private fields such as 'apk'. |
254 |
Unnecessary use of fully qualified name 'AndroidExtension.APK' due to existing static import 'com.simpligility.maven.plugins.android.common.AndroidExtension.APK' |
381 |
Avoid unused method parameters such as 'sourceFolders'. |
591 |
These nested if statements could be combined |
830–833 |
These nested if statements could be combined |
968–976 |
Avoid empty catch blocks |
993–996 |
These nested if statements could be combined |
1039–1048 |
Unnecessary use of fully qualified name 'org.apache.commons.io.FileUtils.copyDirectory' due to existing import 'org.apache.commons.io.FileUtils' |
1220–1221 |
Avoid unused private methods such as 'getDefaultMetaInf()'. |
1303 |
com/simpligility/maven/plugins/android/phase09package/ApklibMojo.java
Violation |
Line |
Avoid unused private fields such as 'ndkClassifier'. |
93 |
Unnecessary use of fully qualified name 'AndroidExtension.APKLIB' due to existing static import 'com.simpligility.maven.plugins.android.common.AndroidExtension.APKLIB' |
124 |
com/simpligility/maven/plugins/android/phase_prebuild/ClasspathModifierLifecycleParticipant.java
Violation |
Line |
These nested if statements could be combined |
190–194 |
These nested if statements could be combined |
340–346 |
com/simpligility/maven/plugins/android/phase_prebuild/ProvidedDependencyChecker.java
Violation |
Line |
Useless parentheses. |
27 |
Useless parentheses. |
28 |
Useless parentheses. |
29 |
Useless parentheses. |
30 |
com/simpligility/maven/plugins/android/standalonemojos/DeployApkMojo.java
Violation |
Line |
Avoid unused private fields such as 'deployapkFilename'. |
58 |
com/simpligility/maven/plugins/android/standalonemojos/LintMojo.java
Violation |
Line |
Avoid empty if statements |
591–595 |
Avoid empty if statements |
610–614 |
Avoid empty if statements |
616–620 |
Avoid empty if statements |
622–626 |
com/simpligility/maven/plugins/android/standalonemojos/ManifestMergerMojo.java
Violation |
Line |
Useless parentheses. |
297 |
com/simpligility/maven/plugins/android/standalonemojos/ManifestUpdateMojo.java
Violation |
Line |
Useless parentheses. |
543 |
Useless parentheses. |
544 |
Useless parentheses. |
545 |
com/simpligility/maven/plugins/android/standalonemojos/MonkeyRunnerMojo.java
Violation |
Line |
An operation on an Immutable object (String, BigDecimal or BigInteger) wont change the object itself |
305 |
com/simpligility/maven/plugins/android/standalonemojos/PullMojo.java
Violation |
Line |
Avoid unused private fields such as 'pull'. |
75 |
Avoid unused private fields such as 'pullSource'. |
81 |
Avoid unused private fields such as 'pullDestination'. |
100 |
com/simpligility/maven/plugins/android/standalonemojos/RedeployApkMojo.java
Violation |
Line |
Avoid unused private fields such as 'deployapkFilename'. |
60 |
com/simpligility/maven/plugins/android/standalonemojos/RunMojo.java
Violation |
Line |
Avoid unused private fields such as 'run'. |
122 |
com/simpligility/maven/plugins/android/standalonemojos/UndeployApkMojo.java
Violation |
Line |
Avoid unused private fields such as 'deployapkFilename'. |
61 |
Avoid unused private fields such as 'deployapkPackagename'. |
67 |
com/simpligility/maven/plugins/android/standalonemojos/UnpackMojo.java
Violation |
Line |
Avoid unused private fields such as 'unpackLazy'. |
68 |
Avoid unused private fields such as 'unpack'. |
75 |
com/simpligility/maven/plugins/android/standalonemojos/ZipalignMojo.java
Violation |
Line |
Avoid unused private fields such as 'zipalign'. |
54 |
Avoid unused private fields such as 'zipalignSkip'. |
61 |
Avoid unused private fields such as 'zipalignVerbose'. |
71 |
Avoid unused private fields such as 'zipalignInputApk'. |
83 |
Avoid unused private fields such as 'zipalignOutputApk'. |
95 |
Avoid unused private fields such as 'zipalignClassifier'. |
113 |
Avoid unused private methods such as 'getInputApkPath()'. |
236 |
Avoid unused private methods such as 'getOutputApkPath()'. |
252 |
com/simpligility/maven/plugins/android/standalonemojos/HelpMojo.java
Violation |
Line |
Useless parentheses. |
238 |
Useless parentheses. |
283 |
Useless parentheses. |
293 |