Checkstyle Results
The following document contains the results of Checkstyle 6.18 with /Users/rgoers/projects/apache/logging/log4j/log4j-2.17.1/log4j-1.2-api/../checkstyle.xml ruleset. 
Summary
| Files | |||
|---|---|---|---|
| 101 | 0 | 0 | 293 |
Files
Details
org/apache/log4j/AppenderSkeleton.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 29 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 31 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 33 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 35 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 39 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 41 | |
| whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 58 | |
| coding | SuperFinalize | Method 'finalize' should call 'super.finalize'. | 74 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 144 |
org/apache/log4j/Category.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'org.apache.logging.log4j.message.LocalizedMessage' import. | 36 | |
| naming | ConstantName | Name 'isCoreAvailable' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$|^capabilities$|^log$'. | 55 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 73 | |
| coding | HiddenField | 'bundle' hides a field. | 419 | |
| blocks | RightCurly | '}' at column 13 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else or try/catch/finally). | 520 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 540 |
org/apache/log4j/Layout.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 27 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 27 |
org/apache/log4j/LogManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'isLog4jCore' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$|^capabilities$|^log$'. | 71 |
org/apache/log4j/Priority.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'level' must be private and have accessor methods. | 96 | |
| design | VisibilityModifier | Variable 'levelStr' must be private and have accessor methods. | 97 | |
| design | VisibilityModifier | Variable 'syslogEquivalent' must be private and have accessor methods. | 98 |
org/apache/log4j/WriterAppender.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.io.IOException' import. | 25 | |
| javadoc | JavadocMethod | Expected @param tag for 'layout'. | 80 | |
| javadoc | JavadocMethod | Expected @param tag for 'os'. | 80 | |
| javadoc | JavadocMethod | Expected @param tag for 'layout'. | 91 | |
| javadoc | JavadocMethod | Expected @param tag for 'writer'. | 91 |
org/apache/log4j/bridge/AppenderAdapter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| javadoc | JavadocType | Missing a Javadoc comment. | 66 |
org/apache/log4j/bridge/AppenderWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 93 |
org/apache/log4j/bridge/LayoutAdapter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.HashMap' import. | 23 |
org/apache/log4j/bridge/LayoutWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 35 |
org/apache/log4j/bridge/LogEventAdapter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.lang.reflect.Method' import. | 31 |
org/apache/log4j/bridge/LogEventWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.HashMap' import. | 38 | |
| coding | HiddenField | 'thread' hides a field. | 131 | |
| coding | HiddenField | 'thread' hides a field. | 137 | |
| coding | HiddenField | 'thread' hides a field. | 143 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 189 | |
| whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 209 | |
| whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 214 |
org/apache/log4j/bridge/RewritePolicyAdapter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocMethod | Expected @param tag for 'policy'. | 34 |
org/apache/log4j/bridge/RewritePolicyWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'org.apache.log4j.rewrite.RewritePolicy' import. | 21 |
org/apache/log4j/builders/AbstractBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| imports | ImportOrder | Wrong order for 'java.util.ArrayList' import. | 31 | |
| naming | StaticVariableName | Name 'LOGGER' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 42 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 43 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 43 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 44 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 44 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 45 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 45 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 46 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 46 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 47 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 47 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 48 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 48 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 49 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 49 |
org/apache/log4j/builders/BuilderManager.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.lang.reflect.Constructor' import. | 36 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 46 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 49 | |
| whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 49 | |
| sizes | LineLength | Line is longer than 120 characters (found 134). | 141 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 164 |
org/apache/log4j/builders/Holder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'value' hides a field. | 38 |
org/apache/log4j/builders/appender/AppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 24 |
org/apache/log4j/builders/appender/AsyncAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| imports | ImportOrder | Wrong order for 'java.util.ArrayList' import. | 27 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 49 | |
| coding | MagicNumber | '1024' is a magic number. | 73 | |
| coding | MagicNumber | '1024' is a magic number. | 136 |
org/apache/log4j/builders/appender/ConsoleAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.ArrayList' import. | 28 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 49 | |
| sizes | LineLength | Line is longer than 120 characters (found 125). | 97 |
org/apache/log4j/builders/appender/DailyRollingFileAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 28 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 54 | |
| coding | MagicNumber | '8192' is a magic number. | 80 | |
| whitespace | WhitespaceAround | '+' is not followed by whitespace. | 172 |
org/apache/log4j/builders/appender/FileAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 28 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 49 | |
| coding | MagicNumber | '8192' is a magic number. | 74 |
org/apache/log4j/builders/appender/NullAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 29 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 33 |
org/apache/log4j/builders/appender/RewriteAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.ArrayList' import. | 28 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 53 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 108 |
org/apache/log4j/builders/appender/RollingFileAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 28 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 56 | |
| coding | MagicNumber | '8192' is a magic number. | 80 | |
| whitespace | WhitespaceAround | '+' is not followed by whitespace. | 193 |
org/apache/log4j/builders/appender/SyslogAppenderBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.io.Serializable' import. | 28 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 52 | |
| naming | StaticVariableName | Name 'DEFAULT_PORT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 59 | |
| coding | MagicNumber | '514' is a magic number. | 59 | |
| sizes | LineLength | Line is longer than 120 characters (found 129). | 119 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 133 | |
| sizes | LineLength | Line is longer than 120 characters (found 121). | 137 |
org/apache/log4j/builders/filter/DenyAllFilterBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocStyle | First sentence should end with a period. | 31 |
org/apache/log4j/builders/filter/LevelMatchFilterBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 24 |
org/apache/log4j/builders/filter/LevelRangeFilterBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 24 |
org/apache/log4j/builders/filter/StringMatchFilterBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.concurrent.atomic.AtomicBoolean' import. | 24 |
org/apache/log4j/builders/layout/HtmlLayoutBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 23 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 38 |
org/apache/log4j/builders/layout/PatternLayoutBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 34 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 39 | |
| whitespace | NoWhitespaceAfter | '++' is followed by whitespace. | 61 |
org/apache/log4j/builders/layout/SimpleLayoutBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| javadoc | JavadocStyle | First sentence should end with a period. | 31 |
org/apache/log4j/builders/layout/TTCCLayoutBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 25 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 41 |
org/apache/log4j/builders/layout/XmlLayoutBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 23 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 37 |
org/apache/log4j/builders/rewrite/RewritePolicyBuilder.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. |
org/apache/log4j/config/InputStreamWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocType | Missing a Javadoc comment. | 22 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 27 |
org/apache/log4j/config/Log4j1Configuration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 31 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 32 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 33 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 35 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 39 |
org/apache/log4j/config/Log4j1ConfigurationConverter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 126). | 57 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 62 | |
| coding | DeclarationOrder | Instance variable definition in wrong order. | 148 | |
| coding | DeclarationOrder | Constructor definition in wrong order. | 150 | |
| sizes | LineLength | Line is longer than 120 characters (found 127). | 186 |
org/apache/log4j/config/Log4j1ConfigurationFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 40 |
org/apache/log4j/config/Log4j1ConfigurationParser.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 139). | 56 | |
| sizes | LineLength | Line is longer than 120 characters (found 139). | 57 | |
| sizes | LineLength | Line is longer than 120 characters (found 133). | 58 | |
| sizes | LineLength | Line is longer than 120 characters (found 140). | 59 | |
| sizes | LineLength | Line is longer than 120 characters (found 124). | 60 |
org/apache/log4j/config/PropertiesConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.apache.log4j.LogManager. | 35 | |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 60 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 63 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 63 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 64 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 64 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 64 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 64 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 597 | |
| coding | MultipleVariableDeclarations | Each variable declaration must be in its own statement. | 598 | |
| design | VisibilityModifier | Variable 'key' must be private and have accessor methods. | 598 | |
| design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 598 |
org/apache/log4j/config/PropertiesConfigurationFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 40 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 45 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 49 | |
| whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 52 |
org/apache/log4j/config/PropertySetter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| naming | StaticVariableName | Name 'LOGGER' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 60 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 61 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
org/apache/log4j/helpers/AppenderAttachableImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Collections' import. | 23 |
org/apache/log4j/helpers/OptionConverter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.io.InterruptedIOException' import. | 26 | |
| design | FinalClass | Class OptionConverter should be declared as final. | 33 | |
| naming | StaticVariableName | Name 'DELIM_START' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 35 | |
| design | VisibilityModifier | Variable 'DELIM_START' must be private and have accessor methods. | 35 | |
| naming | StaticVariableName | Name 'DELIM_STOP' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 36 | |
| design | VisibilityModifier | Variable 'DELIM_STOP' must be private and have accessor methods. | 36 | |
| naming | StaticVariableName | Name 'DELIM_START_LEN' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 37 | |
| design | VisibilityModifier | Variable 'DELIM_START_LEN' must be private and have accessor methods. | 37 | |
| naming | StaticVariableName | Name 'DELIM_STOP_LEN' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 38 | |
| design | VisibilityModifier | Variable 'DELIM_STOP_LEN' must be private and have accessor methods. | 38 | |
| naming | ConstantName | Name 'charMap' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$|^capabilities$|^log$'. | 40 | |
| whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 191 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 369 | |
| design | VisibilityModifier | Variable 'key' must be private and have accessor methods. | 370 | |
| design | VisibilityModifier | Variable 'replacement' must be private and have accessor methods. | 371 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 373 |
org/apache/log4j/helpers/QuietWriter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.io.FilterWriter' import. | 23 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 33 |
org/apache/log4j/layout/Log4j1XmlLayout.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| coding | DeclarationOrder | Constructor definition in wrong order. | 58 | |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 112 |
org/apache/log4j/legacy/core/CategoryUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. |
org/apache/log4j/or/ObjectRenderer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| javadoc | JavadocMethod | Expected an @return tag. | 26 | |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 26 | |
| javadoc | JavadocMethod | Expected @param tag for 'o'. | 26 |
org/apache/log4j/or/ThreadGroupRenderer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | WhitespaceAround | 'if' is not followed by whitespace. | 28 | |
| whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 53 |
org/apache/log4j/or/jms/MessageRenderer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| imports | ImportOrder | Wrong order for 'javax.jms.Message' import. | 23 | |
| imports | ImportOrder | Wrong order for 'javax.jms.JMSException' import. | 24 | |
| imports | ImportOrder | Wrong order for 'javax.jms.DeliveryMode' import. | 25 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 27 | |
| whitespace | WhitespaceAround | 'switch' is not followed by whitespace. | 44 | |
| whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 80 |
org/apache/log4j/pattern/Log4j1MdcPatternConverter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 120 characters (found 177). | 82 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 82 | |
| naming | StaticVariableName | Name 'APPEND_EACH' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 82 |
org/apache/log4j/pattern/Log4j1NdcPatternConverter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| imports | ImportOrder | Wrong order for 'java.util.List' import. | 26 |
org/apache/log4j/rewrite/MapRewritePolicy.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.util.HashMap' import. | 30 |
org/apache/log4j/rewrite/PropertyRewritePolicy.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.util.Collections' import. | 28 |
org/apache/log4j/rewrite/RewritePolicy.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 |
org/apache/log4j/spi/AppenderAttachable.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.util.Enumeration' import. | 22 |
org/apache/log4j/spi/Configurator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 29 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 29 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 31 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 31 | |
| javadoc | JavadocMethod | Expected @param tag for 'loggerContext'. | 44 | |
| javadoc | JavadocMethod | Expected @param tag for 'loggerContext'. | 54 |
org/apache/log4j/spi/ErrorCode.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| design | InterfaceIsType | interfaces should describe a type and hence have methods. | 24 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 26 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 26 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 27 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 27 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 28 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 28 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 29 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 29 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 30 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 30 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 31 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 31 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 32 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 32 |
org/apache/log4j/spi/Filter.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | WhitespaceAround | 'catch' is not followed by whitespace. | 32 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 42 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 48 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 54 | |
| coding | DeclarationOrder | Instance variable definition in wrong order. | 61 | |
| design | VisibilityModifier | Variable 'next' must be private and have accessor methods. | 62 |
org/apache/log4j/spi/LocationInfo.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 28 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 28 | |
| design | VisibilityModifier | Variable 'fullInfo' must be private and have accessor methods. | 28 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 38 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 38 | |
| coding | DeclarationOrder | Static variable definition in wrong order. | 40 | |
| javadoc | JavadocMethod | Expected an @return tag. | 47 | |
| javadoc | JavadocMethod | Expected an @return tag. | 55 | |
| javadoc | JavadocMethod | Expected an @return tag. | 63 | |
| javadoc | JavadocMethod | Expected an @return tag. | 71 |
org/apache/log4j/spi/LoggingEvent.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | ImportOrder | Wrong order for 'java.util.Map' import. | 23 |
org/apache/log4j/xml/Log4jEntityResolver.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 |
org/apache/log4j/xml/UnrecognizedElementHandler.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | ||
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| imports | ImportOrder | Wrong order for 'java.util.Properties' import. | 20 |
org/apache/log4j/xml/XmlConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 70 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 70 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 71 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 71 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 76 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 76 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 78 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 78 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 82 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 82 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 84 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 84 | |
| whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 89 | |
| naming | ConstantName | Name 'dbfKey' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$|^capabilities$|^log$'. | 90 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 93 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 93 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 98 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 103 | |
| coding | HiddenField | 'props' hides a field. | 202 | |
| coding | HiddenField | 'props' hides a field. | 225 | |
| coding | HiddenField | 'props' hides a field. | 245 | |
| coding | HiddenField | 'props' hides a field. | 262 | |
| coding | HiddenField | 'props' hides a field. | 283 | |
| javadoc | JavadocMethod | Expected an @return tag. | 349 | |
| javadoc | JavadocMethod | Expected @param tag for 'appenderRef'. | 349 | |
| javadoc | JavadocMethod | Expected an @return tag. | 358 | |
| javadoc | JavadocMethod | Expected @param tag for 'appenderElement'. | 358 | |
| javadoc | JavadocMethod | Expected an @return tag. | 497 | |
| javadoc | JavadocMethod | Expected @param tag for 'filterElement'. | 497 | |
| javadoc | JavadocMethod | Expected an @return tag. | 581 | |
| javadoc | JavadocMethod | Expected @param tag for 'layoutElement'. | 581 | |
| naming | ParameterName | Name 'layout_element' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 591 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 719 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 732 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 767 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 771 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 795 |
org/apache/log4j/xml/XmlConfigurationFactory.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| header | Header | Line does not match expected header line of ' * contributor license agreements. See the NOTICE file distributed with'. | 3 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 44 | |
| coding | DeclarationOrder | Variable access definition in wrong order. | 49 | |
| sizes | LineLength | Line is longer than 120 characters (found 122). | 53 | |
| whitespace | NoWhitespaceAfter | '{' is followed by whitespace. | 56 |





