PMD Results
The following document contains the results of PMD 3.7.
Files
org/apache/geronimo/system/jmx/KernelMBeanServer.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 142 - 144 | 
| Avoid empty catch blocks | 148 - 150 | 
| Avoid empty catch blocks | 229 - 231 | 
| Avoid empty catch blocks | 248 - 250 | 
| Avoid empty catch blocks | 282 - 284 | 
| Avoid empty catch blocks | 288 - 290 | 
org/apache/geronimo/system/jmx/MBeanServerKernelBridge.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 90 - 92 | 
| Avoid empty catch blocks | 108 - 110 | 
| Avoid empty catch blocks | 110 - 112 | 
| Avoid empty catch blocks | 126 - 129 | 
org/apache/geronimo/system/sharedlib/SharedLib.java
| Violation | Line | 
|---|
| These nested if statements could be combined | 51 - 53 | 
| These nested if statements could be combined | 70 - 72 | 
org/apache/geronimo/system/main/LocalServer.java
| Violation | Line | 
|---|
| Overriding method merely calls super | 48 - 50 | 
org/apache/geronimo/system/main/Daemon.java
| Violation | Line | 
|---|
| Avoid unused private methods such as 'JVMCheck()' | 185 | 
| Avoid empty catch blocks | 248 - 250 | 
| Avoid unused private methods such as 'AddToSystemProperty(String,List,File)' | 337 | 
org/apache/geronimo/system/main/SilentStartupMonitor.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 55 - 56 | 
org/apache/geronimo/system/main/StartupMonitorUtil.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 122 - 124 | 
| These nested if statements could be combined | 255 - 257 | 
org/apache/geronimo/system/main/CommandLine.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 98 - 100 | 
| Avoid empty catch blocks | 138 - 140 | 
org/apache/geronimo/system/logging/log4j/URLConfigurator.java
| Violation | Line | 
|---|
| These nested if statements could be combined | 64 - 66 | 
org/apache/geronimo/system/logging/log4j/Log4jService.java
| Violation | Line | 
|---|
| Avoid unused imports such as 'org.apache.geronimo.system.serverinfo.ServerConstants' | 55 | 
| These nested if statements could be combined | 320 - 322 | 
| Avoid empty catch blocks | 419 | 
| Avoid empty catch blocks | 671 - 672 | 
org/apache/geronimo/system/configuration/ExecutableConfigurationUtil.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 199 - 200 | 
| Avoid empty catch blocks | 208 - 209 | 
| Avoid empty catch blocks | 217 - 218 | 
| Avoid empty catch blocks | 226 - 227 | 
| Avoid empty catch blocks | 235 - 236 | 
org/apache/geronimo/system/configuration/SieveEncodingInfo.java
| Violation | Line | 
|---|
| These nested if statements could be combined | 69 - 70 | 
org/apache/geronimo/system/configuration/LocalAttributeManager.java
| Violation | Line | 
|---|
| These nested if statements could be combined | 464 - 466 | 
| These nested if statements could be combined | 471 - 473 | 
| Avoid empty catch blocks | 520 - 522 | 
| These nested if statements could be combined | 650 - 652 | 
org/apache/geronimo/system/configuration/OutputFormat.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 883 | 
| Avoid empty catch blocks | 907 | 
org/apache/geronimo/system/configuration/RepositoryConfigurationStore.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 193 - 194 | 
| Avoid empty catch blocks | 331 - 333 | 
| Avoid empty catch blocks | 344 - 346 | 
org/apache/geronimo/system/configuration/ServerOverride.java
| Violation | Line | 
|---|
| Avoid unused imports such as 'java.io.PrintWriter' | 19 | 
org/apache/geronimo/system/configuration/BaseMarkupSerializer.java
| Violation | Line | 
|---|
| These nested if statements could be combined | 875 - 877 | 
| Avoid empty catch blocks | 959 - 961 | 
| Avoid empty catch blocks | 968 - 970 | 
org/apache/geronimo/system/configuration/ConfigurationStoreUtil.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 64 - 65 | 
| Avoid empty catch blocks | 116 - 117 | 
| Avoid empty catch blocks | 158 - 159 | 
org/apache/geronimo/system/configuration/InPlaceConfigurationUtil.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 65 - 67 | 
| Avoid empty catch blocks | 89 - 90 | 
org/apache/geronimo/system/repository/AbstractRepository.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 159 - 161 | 
| Avoid empty catch blocks | 210 - 212 | 
org/apache/geronimo/system/repository/CopyArtifactTypeHandler.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 75 - 76 | 
| Avoid empty catch blocks | 81 - 82 | 
org/apache/geronimo/system/rmi/RMIClassLoaderSpiImpl.java
| Violation | Line | 
|---|
| Avoid unused local variables such as 'url' | 145 | 
| Avoid empty catch blocks | 149 - 151 | 
| Avoid empty catch blocks | 194 | 
org/apache/geronimo/system/plugin/PluginInstallerGBean.java
| Violation | Line | 
|---|
| Avoid empty if statements | 281 | 
| These nested if statements could be combined | 313 - 315 | 
| Avoid empty catch blocks | 333 - 335 | 
| These nested if statements could be combined | 616 - 629 | 
| These nested if statements could be combined | 797 - 800 | 
| Avoid empty catch blocks | 852 | 
| Avoid empty catch blocks | 857 | 
| These nested if statements could be combined | 935 - 937 | 
| Avoid unused method parameters such as 'repo' | 1261 - 1285 | 
org/apache/geronimo/system/plugin/SnapshotVersion.java
| Violation | Line | 
|---|
| These nested if statements could be combined | 67 - 76 | 
org/apache/geronimo/system/plugin/PluginRepositoryDownloader.java
| Violation | Line | 
|---|
| Avoid empty catch blocks | 119 |