PMD ResultsThe following document contains the results of PMD 5.0.2. Filesorg/apache/logging/log4j/core/LoggerContext.java| Violation | Line |
|---|
| Useless parentheses. | 147 |
org/apache/logging/log4j/core/appender/AsyncAppender.java| Violation | Line |
|---|
| Useless parentheses. | 147 | | Useless parentheses. | 147 | | Avoid empty catch blocks | 231 - 233 | | Avoid empty catch blocks | 238 - 240 |
org/apache/logging/log4j/core/appender/FailoverAppender.java| Violation | Line |
|---|
| Useless parentheses. | 133 |
org/apache/logging/log4j/core/appender/SyslogAppender.java| Violation | Line |
|---|
| Useless parentheses. | 127 - 131 |
org/apache/logging/log4j/core/appender/TLSSyslogAppender.java| Violation | Line |
|---|
| Useless parentheses. | 121 - 125 |
org/apache/logging/log4j/core/appender/TLSSyslogFrame.java| Violation | Line |
|---|
| Overriding method merely calls super | 57 - 59 | | Ensure you override both equals() and hashCode() | 57 |
org/apache/logging/log4j/core/appender/db/jdbc/ColumnConfig.java| Violation | Line |
|---|
| Useless parentheses. | 120 | | Useless parentheses. | 120 | | Useless parentheses. | 120 |
org/apache/logging/log4j/core/appender/db/jdbc/JDBCDatabaseManager.java| Violation | Line |
|---|
| Avoid unused private fields such as 'layout'. | 189 |
org/apache/logging/log4j/core/appender/db/jpa/converter/StackTraceElementAttributeConverter.java| Violation | Line |
|---|
| Avoid empty catch blocks | 71 - 73 |
org/apache/logging/log4j/core/appender/rewrite/RewriteAppender.java| Violation | Line |
|---|
| Overriding method merely calls super | 74 - 76 |
org/apache/logging/log4j/core/appender/rolling/OnStartupTriggeringPolicy.java| Violation | Line |
|---|
| Avoid unused private fields such as 'LOGGER'. | 37 |
org/apache/logging/log4j/core/appender/rolling/PatternProcessor.java| Violation | Line |
|---|
| Useless parentheses. | 159 | | Avoid unused private methods such as 'patternContains(String,char...)'. | 237 |
org/apache/logging/log4j/core/appender/rolling/helper/FileRenameAction.java| Violation | Line |
|---|
| These nested if statements could be combined | 80 - 83 |
org/apache/logging/log4j/core/appender/routing/Route.java| Violation | Line |
|---|
| Avoid unused local variables such as 'child'. | 101 |
org/apache/logging/log4j/core/async/AsyncLoggerConfig.java| Violation | Line |
|---|
| Avoid unused imports such as 'org.apache.logging.log4j.Logger' | 24 | | Avoid unused imports such as 'org.apache.logging.log4j.status.StatusLogger' | 37 |
org/apache/logging/log4j/core/async/DaemonThreadFactory.java| Violation | Line |
|---|
| Avoid using java.lang.ThreadGroup; it is not thread safe | 35 |
org/apache/logging/log4j/core/config/AppenderControl.java| Violation | Line |
|---|
| These nested if statements could be combined | 74 - 76 |
org/apache/logging/log4j/core/config/BaseConfiguration.java| Violation | Line |
|---|
| Avoid unused imports such as 'java.util.HashMap' | 26 |
org/apache/logging/log4j/core/config/ConfigurationFactory.java| Violation | Line |
|---|
| Avoid empty catch blocks | 285 - 287 | | Useless parentheses. | 316 | | Avoid empty catch blocks | 317 - 319 | | Avoid empty catch blocks | 375 - 377 | | Useless parentheses. | 466 |
org/apache/logging/log4j/core/config/Configurator.java| Violation | Line |
|---|
| Avoid empty catch blocks | 137 - 139 |
org/apache/logging/log4j/core/config/JSONConfiguration.java| Violation | Line |
|---|
| Overriding method merely calls super | 162 - 164 |
org/apache/logging/log4j/core/config/JSONConfigurationFactory.java| Violation | Line |
|---|
| Avoid unused private fields such as 'configFile'. | 41 |
org/apache/logging/log4j/core/config/LoggerConfig.java| Violation | Line |
|---|
| Overriding method merely calls super | 150 - 152 |
org/apache/logging/log4j/core/config/XMLConfiguration.java| Violation | Line |
|---|
| Useless parentheses. | 325 |
org/apache/logging/log4j/core/filter/MapFilter.java| Violation | Line |
|---|
| Useless parentheses. | 82 | | Useless parentheses. | 82 |
org/apache/logging/log4j/core/filter/StructuredDataFilter.java| Violation | Line |
|---|
| Useless parentheses. | 74 | | Useless parentheses. | 74 |
org/apache/logging/log4j/core/filter/ThreadContextMapFilter.java| Violation | Line |
|---|
| Useless parentheses. | 98 | | Useless parentheses. | 98 |
org/apache/logging/log4j/core/helpers/Booleans.java| Violation | Line |
|---|
| Useless parentheses. | 34 |
org/apache/logging/log4j/core/helpers/Charsets.java| Violation | Line |
|---|
| These nested if statements could be combined | 58 - 60 |
org/apache/logging/log4j/core/helpers/Closer.java| Violation | Line |
|---|
| Avoid empty catch blocks | 42 - 44 | | Avoid empty catch blocks | 70 - 72 | | Avoid empty catch blocks | 98 - 100 |
org/apache/logging/log4j/core/helpers/FileUtils.java| Violation | Line |
|---|
| Useless parentheses. | 53 - 54 | | Useless parentheses. | 54 |
org/apache/logging/log4j/core/helpers/Throwables.java| Violation | Line |
|---|
| Avoid empty catch blocks | 45 - 47 |
org/apache/logging/log4j/core/helpers/Transform.java| Violation | Line |
|---|
| Useless parentheses. | 48 - 51 | | Useless parentheses. | 126 - 133 |
org/apache/logging/log4j/core/helpers/UUIDUtil.java| Violation | Line |
|---|
| Avoid empty catch blocks | 99 - 101 | | Useless parentheses. | 180 | | Useless parentheses. | 180 - 181 | | Useless parentheses. | 181 |
org/apache/logging/log4j/core/impl/Log4jLogEvent.java| Violation | Line |
|---|
| Avoid unused method parameters such as 'stream'. | 385 |
org/apache/logging/log4j/core/impl/ThrowableFormatOptions.java| Violation | Line |
|---|
| Avoid empty if statements | 207 - 209 |
org/apache/logging/log4j/core/impl/ThrowableProxy.java| Violation | Line |
|---|
| Avoid empty catch blocks | 434 - 436 | | Avoid empty catch blocks | 463 - 465 | | Avoid empty catch blocks | 501 - 503 |
org/apache/logging/log4j/core/jmx/Server.java| Violation | Line |
|---|
| Avoid unused method parameters such as 'executor'. | 174 | | Avoid unused method parameters such as 'executor'. | 218 | | Avoid unused method parameters such as 'executor'. | 229 |
org/apache/logging/log4j/core/layout/HTMLLayout.java| Violation | Line |
|---|
| Avoid empty catch blocks | 219 - 221 |
org/apache/logging/log4j/core/layout/JSONLayout.java| Violation | Line |
|---|
| Useless parentheses. | 317 |
org/apache/logging/log4j/core/layout/RFC5424Layout.java| Violation | Line |
|---|
| Avoid unused private fields such as 'eventPrefix'. | 104 | | Useless parentheses. | 349 | | Useless parentheses. | 491 | | Avoid unused private methods such as 'getId(StructuredDataId)'. | 505 |
org/apache/logging/log4j/core/layout/XMLLayout.java| Violation | Line |
|---|
| Useless parentheses. | 317 |
org/apache/logging/log4j/core/lookup/StrSubstitutor.java| Violation | Line |
|---|
| Useless parentheses. | 759 | | Useless parentheses. | 835 | | Useless parentheses. | 835 | | Useless parentheses. | 1119 |
org/apache/logging/log4j/core/lookup/WebLookup.java| Violation | Line |
|---|
| No need to import a type that lives in the same package | 25 |
org/apache/logging/log4j/core/net/MulticastDNSAdvertiser.java| Violation | Line |
|---|
| Avoid empty catch blocks | 90 - 92 | | Avoid empty catch blocks | 236 - 238 |
org/apache/logging/log4j/core/net/Priority.java| Violation | Line |
|---|
| Useless parentheses. | 46 |
org/apache/logging/log4j/core/net/SocketServer.java| Violation | Line |
|---|
| Avoid empty catch blocks | 193 - 195 | | Avoid empty catch blocks | 223 - 225 | | Avoid empty catch blocks | 230 - 232 | | Avoid empty catch blocks | 232 - 234 | | Avoid empty catch blocks | 241 - 243 |
org/apache/logging/log4j/core/net/TCPSocketManager.java| Violation | Line |
|---|
| Avoid empty catch blocks | 201 - 203 |
org/apache/logging/log4j/core/net/UDPSocketServer.java| Violation | Line |
|---|
| Avoid empty catch blocks | 169 - 171 | | Avoid empty catch blocks | 176 - 178 | | Avoid empty catch blocks | 178 - 180 | | Avoid empty catch blocks | 187 - 189 |
org/apache/logging/log4j/core/net/ssl/KeyStoreConfiguration.java| Violation | Line |
|---|
| Avoid unused imports such as 'org.apache.logging.log4j.status.StatusLogger' | 22 | | Avoid empty catch blocks | 81 - 82 |
org/apache/logging/log4j/core/net/ssl/TrustStoreConfiguration.java| Violation | Line |
|---|
| Avoid unused imports such as 'org.apache.logging.log4j.status.StatusLogger' | 22 |
org/apache/logging/log4j/core/pattern/AnsiEscape.java| Violation | Line |
|---|
| Avoid empty catch blocks | 325 - 327 |
org/apache/logging/log4j/core/pattern/CachedDateFormat.java| Violation | Line |
|---|
| Useless parentheses. | 211 | | Useless parentheses. | 268 | | Useless parentheses. | 268 | | Useless parentheses. | 268 | | Useless parentheses. | 380 | | Useless parentheses. | 380 |
org/apache/logging/log4j/core/pattern/MarkerPatternConverter.java| Violation | Line |
|---|
| Avoid unused constructor parameters such as 'options'. | 34 |
org/apache/logging/log4j/core/pattern/PatternParser.java| Violation | Line |
|---|
| Useless parentheses. | 493 |
org/apache/logging/log4j/core/pattern/UUIDPatternConverter.java| Violation | Line |
|---|
| Useless parentheses. | 53 |
org/apache/logging/log4j/core/selector/ClassLoaderContextSelector.java| Violation | Line |
|---|
| Avoid unused imports such as 'org.apache.logging.log4j.core.config.Configuration' | 31 | | Avoid empty catch blocks | 116 - 118 | | Avoid empty catch blocks | 146 - 148 |
org/apache/logging/log4j/core/web/Log4jWebInitializerImpl.java| Violation | Line |
|---|
| Avoid empty catch blocks | 46 - 48 |
|
