PMD Results

The following document contains the results of PMD 3.7.

Files

org/apache/geronimo/kernel/KernelRegistry.java

ViolationLine
Avoid unused private fields such as 'key'117

org/apache/geronimo/kernel/classloader/DirectoryResourceLocation.java

ViolationLine
Avoid empty catch blocks74 - 76

org/apache/geronimo/kernel/classloader/IoUtil.java

ViolationLine
Avoid unused imports such as 'java.io.BufferedOutputStream'19
Avoid unused imports such as 'java.io.File'21
Avoid unused imports such as 'java.io.FileInputStream'22
Avoid unused imports such as 'java.io.FileOutputStream'23
Avoid unused imports such as 'java.io.InputStreamReader'26
Avoid unused imports such as 'java.io.FileNotFoundException'30
Avoid unused imports such as 'java.net.MalformedURLException'31
Avoid unused imports such as 'java.net.URL'32
Avoid unused imports such as 'java.net.URLStreamHandler'33
Avoid unused imports such as 'java.net.URLConnection'34
Avoid unused imports such as 'java.util.Collection'35
Avoid unused imports such as 'java.util.Collections'36
Avoid unused imports such as 'java.util.Enumeration'37
Avoid unused imports such as 'java.util.LinkedList'38
Avoid unused imports such as 'java.util.jar.JarOutputStream'40
Avoid unused imports such as 'java.util.jar.Manifest'41
Avoid unused imports such as 'java.util.zip.ZipEntry'42
Avoid unused imports such as 'java.util.zip.ZipFile'43
Avoid unused imports such as 'org.apache.geronimo.kernel.classloader.ResourceLocation'45
No need to import a type that's in the same package45
Avoid unused imports such as 'org.apache.geronimo.kernel.classloader.JarResourceLocation'46
No need to import a type that's in the same package46
Avoid unused imports such as 'org.apache.geronimo.kernel.classloader.DirectoryResourceLocation'47
No need to import a type that's in the same package47
Avoid empty catch blocks74 - 75
Avoid empty catch blocks83 - 84
Avoid empty catch blocks92 - 93
Avoid empty catch blocks101 - 102
Avoid empty catch blocks110 - 111
Avoid empty catch blocks119 - 120
Avoid empty catch blocks128 - 129

org/apache/geronimo/kernel/classloader/JarResourceLocation.java

ViolationLine
Avoid unused local variables such as 'url'41
Avoid empty catch blocks43 - 44

org/apache/geronimo/kernel/classloader/UrlResourceFinder.java

ViolationLine
Avoid empty catch blocks286 - 288

org/apache/geronimo/kernel/KernelFactory.java

ViolationLine
Avoid empty catch blocks44 - 45
Avoid empty catch blocks61 - 62
Avoid empty catch blocks66 - 67

org/apache/geronimo/kernel/config/KernelConfigurationManager.java

ViolationLine
Avoid empty catch blocks107 - 109
Avoid empty catch blocks225 - 226
Avoid empty catch blocks226 - 227
Avoid empty catch blocks238 - 239
Avoid empty catch blocks239 - 240
Avoid empty catch blocks266 - 268
Avoid empty catch blocks274 - 276
Avoid empty catch blocks311 - 313
Avoid empty catch blocks318 - 320

org/apache/geronimo/kernel/config/ManageableAttributeStore.java

ViolationLine
Avoid unused imports such as 'java.util.Set'28

org/apache/geronimo/kernel/config/SerializedGBeanState.java

ViolationLine
Avoid empty catch blocks114 - 116

org/apache/geronimo/kernel/config/EditableKernelConfigurationManager.java

ViolationLine
Avoid empty catch blocks106 - 108
Avoid empty catch blocks111 - 113
Avoid empty catch blocks116 - 118
Avoid empty catch blocks145 - 147

org/apache/geronimo/kernel/config/ConfigurationUtil.java

ViolationLine
Avoid empty catch blocks93 - 95
Avoid empty catch blocks101 - 103
Avoid empty catch blocks419 - 420
Avoid empty catch blocks420 - 421
Avoid empty catch blocks429 - 430
Avoid empty catch blocks430 - 431

org/apache/geronimo/kernel/config/MultiParentClassLoader.java

ViolationLine
Overriding method merely calls super228 - 231
Avoid empty catch blocks249 - 250
Avoid empty catch blocks262 - 264
Avoid empty catch blocks278 - 279

org/apache/geronimo/kernel/config/SerializedConfigurationMarshaler.java

ViolationLine
Avoid empty catch blocks71 - 72

org/apache/geronimo/kernel/config/xstream/XStreamUtil.java

ViolationLine
Avoid empty catch blocks91 - 93

org/apache/geronimo/kernel/config/xstream/AbstractNameQueryConverter.java

ViolationLine
Avoid unused imports such as 'org.apache.geronimo.gbean.AbstractName'22

org/apache/geronimo/kernel/config/IOUtil.java

ViolationLine
Avoid empty catch blocks128 - 129
Avoid empty catch blocks137 - 138
Avoid empty catch blocks146 - 147
Avoid empty catch blocks155 - 156
Avoid empty catch blocks164 - 165
Avoid empty catch blocks173 - 174
Avoid empty catch blocks182 - 183

org/apache/geronimo/kernel/GBeanNotFoundException.java

ViolationLine
Avoid unused imports such as 'org.apache.geronimo.gbean.AbstractNameQuery'22

org/apache/geronimo/kernel/basic/BasicKernel.java

ViolationLine
Avoid empty catch blocks550

org/apache/geronimo/kernel/basic/BasicProxyMap.java

ViolationLine
Avoid empty while statements207
Avoid unused private fields such as 'next'423
Avoid unused private fields such as 'hashCode'425

org/apache/geronimo/kernel/proxy/ProxyManager.java

ViolationLine
Avoid unused imports such as 'javax.management.ObjectName'21
Avoid unused imports such as 'javax.management.MalformedObjectNameException'22

org/apache/geronimo/kernel/proxy/ProxyFactory.java

ViolationLine
Avoid unused imports such as 'javax.management.ObjectName'22

org/apache/geronimo/kernel/ClassLoading.java

ViolationLine
Avoid empty catch blocks131 - 133

org/apache/geronimo/kernel/repository/WriteableRepository.java

ViolationLine
Avoid unused imports such as 'java.net.URI'19

org/apache/geronimo/kernel/Jsr77Naming.java

ViolationLine
Avoid unused imports such as 'java.util.HashSet'22

org/apache/geronimo/kernel/log/GeronimoLogging.java

ViolationLine
Avoid empty catch blocks68 - 70

org/apache/geronimo/gbean/runtime/GBeanDependency.java

ViolationLine
Unnecessary final modifier in final class72 - 77
Unnecessary final modifier in final class97 - 103
Unnecessary final modifier in final class137 - 148
Unnecessary final modifier in final class150 - 161
Unnecessary final modifier in final class183 - 186

org/apache/geronimo/gbean/runtime/GBeanCollectionReference.java

ViolationLine
These nested if statements could be combined153 - 155

org/apache/geronimo/gbean/runtime/InstanceRegistry.java

ViolationLine
Avoid unused imports such as 'org.apache.geronimo.gbean.AbstractName'19

org/apache/geronimo/gbean/runtime/GBeanInstanceState.java

ViolationLine
Avoid empty catch blocks132 - 134
Avoid empty catch blocks134 - 136
Avoid empty catch blocks183 - 185
Avoid empty catch blocks324 - 326

org/apache/geronimo/gbean/runtime/GBeanInstance.java

ViolationLine
Unnecessary final modifier in final class464 - 466
Unnecessary final modifier in final class468 - 470
Unnecessary final modifier in final class472 - 474
Unnecessary final modifier in final class476 - 478
Unnecessary final modifier in final class484 - 486
Unnecessary final modifier in final class523 - 530
Unnecessary final modifier in final class537 - 544
Unnecessary final modifier in final class550 - 552
Unnecessary final modifier in final class558 - 560
Avoid empty catch blocks780 - 782
These nested if statements could be combined891 - 894