PMD Results
The following document contains the results of PMD 6.4.0.
Files
org/apache/logging/log4j/core/LoggerContext.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 406–408 |
org/apache/logging/log4j/core/appender/AsyncAppenderEventDispatcher.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty while statements | 3 | 156 |
org/apache/logging/log4j/core/appender/ConsoleAppender.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 251 |
org/apache/logging/log4j/core/appender/FailoverAppender.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 133 |
org/apache/logging/log4j/core/appender/FileManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 237 |
| Avoid modifiers which are implied by the context | 3 | 270 |
org/apache/logging/log4j/core/appender/HttpURLConnectionManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty while statements | 3 | 113–115 |
org/apache/logging/log4j/core/appender/MemoryMappedFileAppender.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'org.apache.logging.log4j.core.LogEvent' | 4 | 28 |
org/apache/logging/log4j/core/appender/MemoryMappedFileManager.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 353 |
org/apache/logging/log4j/core/appender/RandomAccessFileAppender.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'org.apache.logging.log4j.core.LogEvent' | 4 | 28 |
org/apache/logging/log4j/core/appender/RollingRandomAccessFileAppender.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'bufferSize'. | 3 | 206 |
org/apache/logging/log4j/core/appender/ScriptAppenderSelector.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'properties'. | 3 | 137 |
org/apache/logging/log4j/core/appender/SmtpAppender.java
| Violation | Priority | Line |
|---|---|---|
| Overriding method merely calls super | 3 | 242–244 |
| Overriding method merely calls super | 3 | 251–253 |
org/apache/logging/log4j/core/appender/db/jdbc/ColumnConfig.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 74 |
| Useless parentheses. | 4 | 74 |
| Useless parentheses. | 4 | 74 |
org/apache/logging/log4j/core/appender/db/jdbc/JdbcDatabaseManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 490–492 |
| Avoid empty catch blocks | 3 | 496–498 |
| Avoid modifiers which are implied by the context | 3 | 661 |
| Avoid empty catch blocks | 3 | 879–881 |
org/apache/logging/log4j/core/appender/mom/JmsManager.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 451–457 |
org/apache/logging/log4j/core/appender/mom/jeromq/JeroMqManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'SHUTDOWN_HOOK'. | 3 | 53 |
org/apache/logging/log4j/core/appender/mom/kafka/KafkaAppender.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 85–87 |
| Avoid empty catch blocks | 3 | 182–184 |
org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
| Violation | Priority | Line |
|---|---|---|
| Ternary operators that can be simplified with || or && | 3 | 126 |
| Useless parentheses. | 4 | 457 |
org/apache/logging/log4j/core/appender/rolling/DirectWriteRolloverStrategy.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'usePrevTime'. | 3 | 239 |
| Useless parentheses. | 4 | 317 |
org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'bufferSize'. | 3 | 95 |
| Ternary operators that can be simplified with || or && | 3 | 733 |
org/apache/logging/log4j/core/appender/rolling/RollingRandomAccessFileManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'immediateFlush'. | 3 | 65 |
org/apache/logging/log4j/core/appender/rolling/action/CommonsCompressAction.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 106 |
| Avoid modifiers which are implied by the context | 3 | 107–109 |
org/apache/logging/log4j/core/appender/rolling/action/Duration.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 121–137 |
| Useless parentheses. | 4 | 167 |
org/apache/logging/log4j/core/appender/rolling/action/FileRenameAction.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 107 |
| Useless parentheses. | 4 | 109 |
org/apache/logging/log4j/core/appender/rolling/action/GzCompressAction.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 135 |
| Avoid modifiers which are implied by the context | 3 | 136 |
| Avoid modifiers which are implied by the context | 3 | 137–138 |
| Avoid modifiers which are implied by the context | 3 | 140 |
org/apache/logging/log4j/core/appender/rolling/action/PathCondition.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 33 |
org/apache/logging/log4j/core/appender/rolling/action/ScriptCondition.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 113–116 |
org/apache/logging/log4j/core/appender/rolling/action/ZipCompressAction.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 98 |
| Avoid modifiers which are implied by the context | 3 | 99 |
org/apache/logging/log4j/core/appender/routing/IdlePurgePolicy.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 78–81 |
org/apache/logging/log4j/core/appender/routing/Routes.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 69 |
| Useless parentheses. | 4 | 69 |
org/apache/logging/log4j/core/async/AsyncLogger.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 344 |
org/apache/logging/log4j/core/async/AsyncLoggerConfig.java
| Violation | Priority | Line |
|---|---|---|
| Overriding method merely calls super | 3 | 125–127 |
org/apache/logging/log4j/core/async/AsyncLoggerConfigDisruptor.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 180 |
org/apache/logging/log4j/core/async/BasicAsyncLoggerContextSelector.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 4 | 24 |
org/apache/logging/log4j/core/async/RingBufferLogEvent.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 229 |
| These nested if statements could be combined | 3 | 323–325 |
| These nested if statements could be combined | 3 | 334–336 |
org/apache/logging/log4j/core/config/ConfigurationFactory.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'HTTPS'. | 3 | 149 |
| Avoid unused private fields such as 'HTTP'. | 3 | 150 |
org/apache/logging/log4j/core/config/ConfigurationScheduler.java
| Violation | Priority | Line |
|---|---|---|
| Overriding method merely calls super | 3 | 55–57 |
org/apache/logging/log4j/core/config/ConfigurationSource.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 90–92 |
| Avoid empty catch blocks | 3 | 223–225 |
| Avoid empty catch blocks | 3 | 237–239 |
org/apache/logging/log4j/core/config/LoggerConfig.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'org.apache.logging.log4j.core.impl.LocationAwareLogEventFactory' | 4 | 46 |
| Overriding method merely calls super | 3 | 167–169 |
org/apache/logging/log4j/core/config/arbiters/ScriptArbiter.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 113–116 |
org/apache/logging/log4j/core/config/json/JsonConfiguration.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused local variables such as 'configFile'. | 3 | 55 |
| Avoid modifiers which are implied by the context | 3 | 58 |
org/apache/logging/log4j/core/config/plugins/convert/Base64Converter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 44–47 |
| Avoid empty catch blocks | 3 | 55–57 |
org/apache/logging/log4j/core/config/plugins/processor/PluginCache.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 67 |
| Avoid modifiers which are implied by the context | 3 | 97 |
org/apache/logging/log4j/core/config/plugins/processor/PluginProcessor.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 117 |
org/apache/logging/log4j/core/config/plugins/util/ResolverUtil.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 220 |
org/apache/logging/log4j/core/config/plugins/visitors/PluginElementVisitor.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty if statements | 3 | 101–103 |
org/apache/logging/log4j/core/config/properties/PropertiesConfigurationFactory.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 47 |
org/apache/logging/log4j/core/config/xml/XmlConfiguration.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused local variables such as 'configFile'. | 3 | 77 |
| Useless parentheses. | 4 | 310 |
org/apache/logging/log4j/core/filter/DenyAllFilter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused method parameters such as 'marker'. | 3 | 64 |
org/apache/logging/log4j/core/filter/MapFilter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 89 |
| Useless parentheses. | 4 | 89 |
| Useless parentheses. | 4 | 102 |
| Useless parentheses. | 4 | 102 |
| Useless parentheses. | 4 | 115 |
| Useless parentheses. | 4 | 115 |
org/apache/logging/log4j/core/filter/ScriptFilter.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 146–149 |
org/apache/logging/log4j/core/filter/StructuredDataFilter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 84 |
| Useless parentheses. | 4 | 84 |
org/apache/logging/log4j/core/filter/ThreadContextMapFilter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 109 |
| Useless parentheses. | 4 | 109 |
org/apache/logging/log4j/core/impl/Log4jContextFactory.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 282 |
org/apache/logging/log4j/core/impl/Log4jLogEvent.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 1023–1025 |
org/apache/logging/log4j/core/impl/ThreadContextDataInjector.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 137 |
org/apache/logging/log4j/core/impl/ThrowableFormatOptions.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 241 |
| Avoid empty if statements | 3 | 257–259 |
org/apache/logging/log4j/core/impl/ThrowableProxyHelper.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 173–175 |
| Avoid empty catch blocks | 3 | 209–211 |
org/apache/logging/log4j/core/jackson/ExtendedStackTraceElementMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'declaringClass'. | 3 | 41 |
| Avoid unused constructor parameters such as 'methodName'. | 3 | 42 |
| Avoid unused constructor parameters such as 'fileName'. | 3 | 43 |
| Avoid unused constructor parameters such as 'lineNumber'. | 3 | 44 |
| Avoid unused constructor parameters such as 'exact'. | 3 | 45 |
| Avoid unused constructor parameters such as 'location'. | 3 | 46 |
| Avoid unused constructor parameters such as 'version'. | 3 | 47 |
org/apache/logging/log4j/core/jackson/Initializers.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 56 |
| Useless parentheses. | 4 | 78 |
org/apache/logging/log4j/core/jackson/InstantMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'epochSecond'. | 3 | 39 |
| Avoid unused constructor parameters such as 'nanoOfSecond'. | 3 | 40 |
org/apache/logging/log4j/core/jackson/Log4jJsonModule.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'objectMessageAsJsonObject'. | 3 | 37 |
org/apache/logging/log4j/core/jackson/Log4jStackTraceElementDeserializer.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty if statements | 3 | 72–74 |
org/apache/logging/log4j/core/jackson/MarkerMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'name'. | 3 | 61 |
org/apache/logging/log4j/core/jackson/StackTraceElementMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'declaringClass'. | 3 | 37 |
| Avoid unused constructor parameters such as 'methodName'. | 3 | 38 |
| Avoid unused constructor parameters such as 'fileName'. | 3 | 39 |
| Avoid unused constructor parameters such as 'lineNumber'. | 3 | 40 |
org/apache/logging/log4j/core/jackson/ThrowableProxyMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'causeProxy'. | 3 | 34 |
| Avoid unused private fields such as 'commonElementCount'. | 3 | 38 |
| Avoid unused private fields such as 'extendedStackTrace'. | 3 | 43 |
| Avoid unused private fields such as 'localizedMessage'. | 3 | 47 |
| Avoid unused private fields such as 'message'. | 3 | 51 |
| Avoid unused private fields such as 'name'. | 3 | 55 |
| Avoid unused private fields such as 'throwable'. | 3 | 58 |
org/apache/logging/log4j/core/jackson/ThrowableProxyWithStacktraceAsStringMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'causeProxy'. | 3 | 33 |
| Avoid unused private fields such as 'commonElementCount'. | 3 | 37 |
| Avoid unused private fields such as 'extendedStackTrace'. | 3 | 40 |
| Avoid unused private fields such as 'localizedMessage'. | 3 | 44 |
| Avoid unused private fields such as 'message'. | 3 | 48 |
| Avoid unused private fields such as 'name'. | 3 | 52 |
| Avoid unused private fields such as 'throwable'. | 3 | 55 |
org/apache/logging/log4j/core/jackson/ThrowableProxyWithoutStacktraceMixIn.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'causeProxy'. | 3 | 33 |
| Avoid unused private fields such as 'commonElementCount'. | 3 | 37 |
| Avoid unused private fields such as 'extendedStackTrace'. | 3 | 40 |
| Avoid unused private fields such as 'localizedMessage'. | 3 | 44 |
| Avoid unused private fields such as 'message'. | 3 | 48 |
| Avoid unused private fields such as 'name'. | 3 | 52 |
| Avoid unused private fields such as 'throwable'. | 3 | 55 |
org/apache/logging/log4j/core/jmx/Server.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused method parameters such as 'executor'. | 3 | 281 |
| Avoid unused method parameters such as 'executor'. | 3 | 355 |
| Avoid unused method parameters such as 'executor'. | 3 | 372 |
| Useless parentheses. | 4 | 380 |
org/apache/logging/log4j/core/layout/ByteBufferDestination.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'org.apache.logging.log4j.core.appender.OutputStreamManager' | 4 | 21 |
org/apache/logging/log4j/core/layout/GelfLayout.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 566 |
| Useless parentheses. | 4 | 643 |
org/apache/logging/log4j/core/layout/HtmlLayout.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 77–103 |
| Avoid empty catch blocks | 3 | 246–248 |
org/apache/logging/log4j/core/layout/LockingStringBuilderEncoder.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private methods such as 'getCharBuffer()'. | 3 | 48 |
| Avoid unused method parameters such as 'destination'. | 3 | 68 |
org/apache/logging/log4j/core/layout/PatternLayout.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private methods such as 'toText(Serializer2,LogEvent,StringBuilder)'. | 3 | 239–240 |
org/apache/logging/log4j/core/layout/Rfc5424Layout.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 338 |
| Useless parentheses. | 4 | 362 |
| Useless parentheses. | 4 | 396 |
| Useless parentheses. | 4 | 511 |
org/apache/logging/log4j/core/layout/ScriptPatternSelector.java
| Violation | Priority | Line |
|---|---|---|
| These nested if statements could be combined | 3 | 86–89 |
org/apache/logging/log4j/core/layout/SerializedLayout.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 67 |
org/apache/logging/log4j/core/layout/StringBuilderEncoder.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused method parameters such as 'destination'. | 3 | 94 |
org/apache/logging/log4j/core/layout/internal/ListChecker.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 25 |
| Avoid modifiers which are implied by the context | 3 | 32–42 |
org/apache/logging/log4j/core/lookup/JndiLookup.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 65 |
org/apache/logging/log4j/core/net/MulticastDnsAdvertiser.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 93–95 |
| Avoid empty catch blocks | 3 | 198–200 |
org/apache/logging/log4j/core/net/SmtpManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 254 |
org/apache/logging/log4j/core/net/ssl/AbstractKeyStoreConfiguration.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 72 |
org/apache/logging/log4j/core/net/ssl/KeyStoreConfiguration.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 162 |
org/apache/logging/log4j/core/net/ssl/TrustStoreConfiguration.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 150 |
org/apache/logging/log4j/core/pattern/AnsiEscape.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 329–331 |
org/apache/logging/log4j/core/pattern/CachedDateFormat.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 211 |
| Useless parentheses. | 4 | 268 |
| Useless parentheses. | 4 | 268 |
| Useless parentheses. | 4 | 268 |
| Useless parentheses. | 4 | 381 |
| Useless parentheses. | 4 | 381 |
org/apache/logging/log4j/core/pattern/EncodingPatternConverter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'java.util.function.Function' | 4 | 20 |
| Avoid unused imports such as 'java.util.stream.Stream' | 4 | 21 |
org/apache/logging/log4j/core/pattern/HighlightConverter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 199 |
org/apache/logging/log4j/core/pattern/HtmlTextRenderer.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'formats'. | 3 | 24 |
org/apache/logging/log4j/core/pattern/JAnsiTextRenderer.java
| Violation | Priority | Line |
|---|---|---|
| Too many static imports may lead to messy code | 3 | 17–332 |
org/apache/logging/log4j/core/pattern/MarkerPatternConverter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'options'. | 3 | 37 |
org/apache/logging/log4j/core/pattern/MarkerSimpleNamePatternConverter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'options'. | 3 | 36 |
org/apache/logging/log4j/core/pattern/MessagePatternConverter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'org.apache.logging.log4j.Logger' | 4 | 23 |
org/apache/logging/log4j/core/pattern/NanoTimePatternConverter.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'options'. | 3 | 37 |
org/apache/logging/log4j/core/pattern/PatternParser.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'expectedClass'. | 3 | 128 |
| These nested if statements could be combined | 3 | 186–188 |
org/apache/logging/log4j/core/pattern/StyleConverter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 99 |
org/apache/logging/log4j/core/pattern/UuidPatternConverter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 53 |
org/apache/logging/log4j/core/pattern/VariablesNotEmptyReplacementConverter.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 86 |
org/apache/logging/log4j/core/script/ScriptFile.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 96–97 |
org/apache/logging/log4j/core/selector/JndiContextSelector.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 146 |
| Avoid modifiers which are implied by the context | 3 | 158 |
org/apache/logging/log4j/core/time/MutableInstant.java
| Violation | Priority | Line |
|---|---|---|
| Too many static imports may lead to messy code | 3 | 17–235 |
org/apache/logging/log4j/core/tools/Generate.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 53–166 |
org/apache/logging/log4j/core/tools/picocli/CommandLine.java
| Violation | Priority | Line |
|---|---|---|
| Avoid modifiers which are implied by the context | 3 | 364–375 |
| Avoid modifiers which are implied by the context | 3 | 387–398 |
| Useless parentheses. | 4 | 1040 |
| Avoid using a branching statement as the last in a loop. | 2 | 1064 |
| Useless parentheses. | 4 | 2088 |
| These nested if statements could be combined | 3 | 2167 |
| Avoid unused method parameters such as 'valueAttachedToOption'. | 3 | 2211 |
| Avoid unused local variables such as 'length'. | 3 | 2216 |
| Ternary operators that can be simplified with || or && | 3 | 2254 |
| Avoid unused method parameters such as 'annotation'. | 3 | 2282 |
| These nested if statements could be combined | 3 | 2316–2318 |
| Avoid unused method parameters such as 'arity'. | 3 | 2325 |
| Avoid unused local variables such as 'desc'. | 3 | 2355 |
| Avoid unused method parameters such as 'cls'. | 3 | 2365 |
| These nested if statements could be combined | 3 | 2431–2433 |
| Avoid unused method parameters such as 'arity'. | 3 | 2441 |
| Avoid unused method parameters such as 'originalSize'. | 3 | 2446 |
| These nested if statements could be combined | 3 | 2491–2493 |
| Useless parentheses. | 4 | 2495 |
| These nested if statements could be combined | 3 | 2539 |
| Useless parentheses. | 4 | 2605 |
| Ternary operators that can be simplified with || or && | 3 | 2965 |
| Ternary operators that can be simplified with || or && | 3 | 2966 |
| Ternary operators that can be simplified with || or && | 3 | 2968 |
| Unnecessary use of fully qualified name 'Help.Column.Overflow.SPAN' due to existing static import 'org.apache.logging.log4j.core.tools.picocli.CommandLine.Help.Column.Overflow.SPAN' | 4 | 3324 |
| Unnecessary use of fully qualified name 'Help.Column.Overflow.WRAP' due to existing static import 'org.apache.logging.log4j.core.tools.picocli.CommandLine.Help.Column.Overflow.WRAP' | 4 | 3325 |
| Useless parentheses. | 4 | 3330 |
| Useless parentheses. | 4 | 3344 |
| Useless parentheses. | 4 | 3699 |
| Avoid unused local variables such as 'o1'. | 3 | 3867 |
| Avoid unused local variables such as 'o2'. | 3 | 3868 |
| Useless parentheses. | 4 | 4274 |
| Useless parentheses. | 4 | 4514 |
| Useless parentheses. | 4 | 4769 |
org/apache/logging/log4j/core/util/Booleans.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 37 |
org/apache/logging/log4j/core/util/CronExpression.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 299 |
| Useless parentheses. | 4 | 299 |
| Useless parentheses. | 4 | 299 |
| These nested if statements could be combined | 3 | 488–491 |
| Useless parentheses. | 4 | 510 |
| Useless parentheses. | 4 | 510 |
| Useless parentheses. | 4 | 510 |
| Useless parentheses. | 4 | 510 |
| Useless parentheses. | 4 | 510 |
| Useless parentheses. | 4 | 580 |
| Useless parentheses. | 4 | 587 |
| Useless parentheses. | 4 | 633 |
| Useless parentheses. | 4 | 635 |
| Useless parentheses. | 4 | 637 |
| Useless parentheses. | 4 | 639 |
| Useless parentheses. | 4 | 778 |
| Useless parentheses. | 4 | 949 |
| Useless parentheses. | 4 | 956 |
| Useless parentheses. | 4 | 962 |
| Useless parentheses. | 4 | 963 |
| Useless parentheses. | 4 | 969 |
| Useless parentheses. | 4 | 975 |
| Useless parentheses. | 4 | 976 |
| Useless parentheses. | 4 | 1141 |
| Unnecessary use of fully qualified name 'java.util.Calendar' due to existing import 'java.util.Calendar' | 4 | 1286 |
| Unnecessary use of fully qualified name 'java.util.Calendar.getInstance' due to existing import 'java.util.Calendar' | 4 | 1286 |
| Unnecessary use of fully qualified name 'java.util.Calendar.getInstance' due to existing import 'java.util.Calendar' | 4 | 1322 |
| Unnecessary use of fully qualified name 'java.util.Calendar' due to existing import 'java.util.Calendar' | 4 | 1322 |
| Unnecessary use of fully qualified name 'java.util.Calendar.HOUR_OF_DAY' due to existing import 'java.util.Calendar' | 4 | 1586 |
| Unnecessary use of fully qualified name 'java.util.Calendar.HOUR_OF_DAY' due to existing import 'java.util.Calendar' | 4 | 1587 |
| Unnecessary use of fully qualified name 'java.util.Calendar.HOUR_OF_DAY' due to existing import 'java.util.Calendar' | 4 | 1588 |
| Useless parentheses. | 4 | 1667 |
| Useless parentheses. | 4 | 1667 |
| Useless parentheses. | 4 | 1676 |
org/apache/logging/log4j/core/util/DefaultShutdownCallbackRegistry.java
| Violation | Priority | Line |
|---|---|---|
| Useless qualified this usage in the same class. | 3 | 110 |
org/apache/logging/log4j/core/util/FileUtils.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 68–70 |
org/apache/logging/log4j/core/util/IOUtils.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused imports such as 'java.io.InputStream' | 4 | 20 |
| Avoid unused imports such as 'java.io.OutputStream' | 4 | 21 |
org/apache/logging/log4j/core/util/JsonUtils.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 97 |
org/apache/logging/log4j/core/util/Loader.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused local variables such as 'className'. | 3 | 334 |
org/apache/logging/log4j/core/util/Log4jThreadFactory.java
| Violation | Priority | Line |
|---|---|---|
| Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 81 |
org/apache/logging/log4j/core/util/NetUtils.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 74–76 |
org/apache/logging/log4j/core/util/Source.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused constructor parameters such as 'lastModified'. | 3 | 66 |
org/apache/logging/log4j/core/util/StringEncoder.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 79 |
| Useless parentheses. | 4 | 79 |
| Useless parentheses. | 4 | 101 |
| Useless parentheses. | 4 | 101 |
| Useless parentheses. | 4 | 102 |
| Useless parentheses. | 4 | 109 |
org/apache/logging/log4j/core/util/Throwables.java
| Violation | Priority | Line |
|---|---|---|
| Avoid empty catch blocks | 3 | 76–78 |
org/apache/logging/log4j/core/util/Transform.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 50–53 |
| Useless parentheses. | 4 | 134–141 |
org/apache/logging/log4j/core/util/TypeUtil.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 218 |
| Useless parentheses. | 4 | 218 |
| Useless parentheses. | 4 | 218 |
org/apache/logging/log4j/core/util/UuidUtil.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'LOGGER'. | 3 | 42 |
| Useless parentheses. | 4 | 145–146 |
| Useless parentheses. | 4 | 145 |
org/apache/logging/log4j/core/util/WatchManager.java
| Violation | Priority | Line |
|---|---|---|
| Avoid unused private fields such as 'NODE_SIZE'. | 3 | 76 |
| Avoid unused private fields such as 'VARIANT'. | 3 | 84 |
| Avoid unused private fields such as 'SEQUENCE_MASK'. | 3 | 85 |
| Useless parentheses. | 4 | 89 |
| Useless parentheses. | 4 | 89–90 |
org/apache/logging/log4j/core/util/datetime/FixedDateFormat.java
| Violation | Priority | Line |
|---|---|---|
| Useless parentheses. | 4 | 367 |
| Useless parentheses. | 4 | 370 |
| Useless parentheses. | 4 | 380 |
| Useless parentheses. | 4 | 383 |
| Useless parentheses. | 4 | 645 |
| Useless parentheses. | 4 | 648 |
| Useless parentheses. | 4 | 654 |
| Useless parentheses. | 4 | 657 |
| Useless parentheses. | 4 | 663 |
| Useless parentheses. | 4 | 664 |
| Useless parentheses. | 4 | 670 |
| Useless parentheses. | 4 | 674 |
| Useless parentheses. | 4 | 677 |
| Useless parentheses. | 4 | 703 |
| Useless parentheses. | 4 | 721 |
| Useless parentheses. | 4 | 721 |