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 |