| Error | Count |
|---|---|
| '0x3f' is a magic number. | 8 |
| 'if' construct must use '{}'s. | 8 |
| Line is longer than 132 characters. | 8 |
| Missing package documentation file. | 8 |
| Redundant 'public' modifier. | 8 |
| ';' is followed by whitespace. | 73 |
| '1024' is a magic number. | 7 |
| 'name' hides a field. | 7 |
| Expected @throws tag for 'MissingAttributeException'. | 7 |
| '0xff' is a magic number. | 6 |
| 'dri' hides a field. | 6 |
| 'var' hides a field. | 6 |
| Expected @param tag for 'sourceBuild'. | 6 |
| Expected @throws tag for 'DependencyResolverException'. | 6 |
| Expected @throws tag for 'IOException'. | 6 |
| Missing a Javadoc comment. | 510 |
| '4' is a magic number. | 5 |
| '5' is a magic number. | 5 |
| '6' is a magic number. | 5 |
| 'file' hides a field. | 5 |
| Expected @param tag for 'context'. | 5 |
| Expected @param tag for 'name'. | 5 |
| More than 7 parameters. | 5 |
| 'static' modifier out of order with the JLS suggestions. | 4 |
| Expected @param tag for 'file'. | 4 |
| '16' is a magic number. | 3 |
| '256' is a magic number. | 3 |
| '26' is a magic number. | 3 |
| '52' is a magic number. | 3 |
| '62' is a magic number. | 3 |
| '8' is a magic number. | 3 |
| 'action' hides a field. | 3 |
| 'basedir' hides a field. | 3 |
| 'context' hides a field. | 3 |
| 'defaultGoalName' hides a field. | 3 |
| 'goalName' hides a field. | 3 |
| 'path' hides a field. | 3 |
| 'project' hides a field. | 3 |
| Expected @param tag for 'jellyScriptHousing'. | 3 |
| Redundant 'final' modifier. | 3 |
| Expected an @return tag. | 27 |
| Must have at least one statement. | 27 |
| Line has trailing spaces. | 267 |
| '10' is a magic number. | 2 |
| '1000' is a magic number. | 2 |
| '12' is a magic number. | 2 |
| '60' is a magic number. | 2 |
| '63' is a magic number. | 2 |
| 'baseContext' hides a field. | 2 |
| 'dependency' hides a field. | 2 |
| 'description' hides a field. | 2 |
| 'goal' hides a field. | 2 |
| 'out' hides a field. | 2 |
| 'parent' hides a field. | 2 |
| 'prereqs' hides a field. | 2 |
| 'session' hides a field. | 2 |
| Expected @param tag for 'cache'. | 2 |
| Expected @param tag for 'cause'. | 2 |
| Expected @param tag for 'directory'. | 2 |
| Expected @param tag for 'goals'. | 2 |
| Expected @param tag for 'jelly'. | 2 |
| Expected @param tag for 'key'. | 2 |
| Expected @param tag for 'message'. | 2 |
| Expected @param tag for 'projects'. | 2 |
| Expected @param tag for 'type'. | 2 |
| Expected @throws tag for 'CyclicGoalChainException'. | 2 |
| Expected @throws tag for 'JellyException'. | 2 |
| Name 'FS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'g1_1' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'g1_2' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'g1_3' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'g2_1' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'g2_2' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'g2_3' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'gc_1' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Name 'gc_2' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2 |
| Expected @throws tag for 'JellyTagException'. | 19 |
| Avoid inline conditionals. | 18 |
| Expected @param tag for 'output'. | 18 |
| '{' is followed by whitespace. | 17 |
| Line does not match expected header line of ' * you may not use this file except in compliance with the License.'. | 136 |
| Comment matches to-do format 'TODO:'. | 13 |
| '3' is a magic number. | 12 |
| Expected @param tag for 'project'. | 12 |
| Expected @throws tag for 'Exception'. | 12 |
| Utility classes should not have a public or default constructor. | 12 |
| Expected @throws tag for 'MavenException'. | 11 |
| '0x0f' is a magic number. | 1 |
| '0xf0' is a magic number. | 1 |
| '1.1' is a magic number. | 1 |
| '18' is a magic number. | 1 |
| '20' is a magic number. | 1 |
| '25' is a magic number. | 1 |
| '32' is a magic number. | 1 |
| '51' is a magic number. | 1 |
| '512' is a magic number. | 1 |
| '9' is a magic number. | 1 |
| 'artifactId' hides a field. | 1 |
| 'artifactList' hides a field. | 1 |
| 'attributes' hides a field. | 1 |
| 'banner' hides a field. | 1 |
| 'cache' hides a field. | 1 |
| 'checksum' hides a field. | 1 |
| 'classLoader' hides a field. | 1 |
| 'collectOnly' hides a field. | 1 |
| 'collectionVar' hides a field. | 1 |
| 'commandLine' hides a field. | 1 |
| 'countedItems' hides a field. | 1 |
| 'countedItemsReversed' hides a field. | 1 |
| 'debug' hides a field. | 1 |
| 'defaultValue' hides a field. | 1 |
| 'dependencyVerifier' hides a field. | 1 |
| 'descriptor' hides a field. | 1 |
| 'else' construct must use '{}'s. | 1 |
| 'emacsMode' hides a field. | 1 |
| 'excludes' hides a field. | 1 |
| 'glob' hides a field. | 1 |
| 'goals' hides a field. | 1 |
| 'handler' hides a field. | 1 |
| 'hashChar' hides a field. | 1 |
| 'housing' hides a field. | 1 |
| 'id' hides a field. | 1 |
| 'ignoreFailures' hides a field. | 1 |
| 'impl' hides a field. | 1 |
| 'includes' hides a field. | 1 |
| 'initialGoal' hides a field. | 1 |
| 'jellyScriptHousing' hides a field. | 1 |
| 'list' hides a field. | 1 |
| 'mapper' hides a field. | 1 |
| 'mavenXml' hides a field. | 1 |
| 'numHashes' hides a field. | 1 |
| 'outputFile' hides a field. | 1 |
| 'overrideType' hides a field. | 1 |
| 'parentContext' hides a field. | 1 |
| 'pluginManager' hides a field. | 1 |
| 'postProcessing' hides a field. | 1 |
| 'projectDescriptor' hides a field. | 1 |
| 'projectList' hides a field. | 1 |
| 'projects' hides a field. | 1 |
| 'requiresAction' hides a field. | 1 |
| 'rootContext' hides a field. | 1 |
| 'rootdir' hides a field. | 1 |
| 'script' hides a field. | 1 |
| 'separator' hides a field. | 1 |
| 'set' hides a field. | 1 |
| 'sort' hides a field. | 1 |
| 'source' hides a field. | 1 |
| 'sourceBuild' hides a field. | 1 |
| 'supportAntVariables' hides a field. | 1 |
| 'tagLibrary' hides a field. | 1 |
| 'tracker' hides a field. | 1 |
| 'useInheritance' hides a field. | 1 |
| 'value' hides a field. | 1 |
| 'while' is not followed by whitespace. | 1 |
| Array brackets at illegal position. | 1 |
| Class InsertionOrderedSetIterator should be declared as final. | 1 |
| Expected @param tag for 'acceptDirectories'. | 1 |
| Expected @param tag for 'artifactId'. | 1 |
| Expected @param tag for 'attributes'. | 1 |
| Expected @param tag for 'attrs'. | 1 |
| Expected @param tag for 'baseDir'. | 1 |
| Expected @param tag for 'build'. | 1 |
| Expected @param tag for 'checksumFileExtension'. | 1 |
| Expected @param tag for 'checksumObserver'. | 1 |
| Expected @param tag for 'child'. | 1 |
| Expected @param tag for 'dependency'. | 1 |
| Expected @param tag for 'destination'. | 1 |
| Expected @param tag for 'goalMapper'. | 1 |
| Expected @param tag for 'handler'. | 1 |
| Expected @param tag for 'housing'. | 1 |
| Expected @param tag for 'id'. | 1 |
| Expected @param tag for 'impl'. | 1 |
| Expected @param tag for 'jarFile'. | 1 |
| Expected @param tag for 'link'. | 1 |
| Expected @param tag for 'localName'. | 1 |
| Expected @param tag for 'manager'. | 1 |
| Expected @param tag for 'mapper'. | 1 |
| Expected @param tag for 'newDebug'. | 1 |
| Expected @param tag for 'newEntry'. | 1 |
| Expected @param tag for 'outputFile'. | 1 |
| Expected @param tag for 'parentProject'. | 1 |
| Expected @param tag for 'path'. | 1 |
| Expected @param tag for 'paths'. | 1 |
| Expected @param tag for 'pluginDirs'. | 1 |
| Expected @param tag for 'pluginFiles'. | 1 |
| Expected @param tag for 'pluginName'. | 1 |
| Expected @param tag for 'pluginSet'. | 1 |
| Expected @param tag for 'postProcessing'. | 1 |
| Expected @param tag for 'properties'. | 1 |
| Expected @param tag for 'rawName'. | 1 |
| Expected @param tag for 'relativePath'. | 1 |
| Expected @param tag for 'remotePath'. | 1 |
| Expected @param tag for 'repositoryPath'. | 1 |
| Expected @param tag for 'resource'. | 1 |
| Expected @param tag for 'session'. | 1 |
| Expected @param tag for 'source'. | 1 |
| Expected @param tag for 'sourceFileName'. | 1 |
| Expected @param tag for 'spe'. | 1 |
| Expected @param tag for 'target'. | 1 |
| Expected @param tag for 'tempDestination'. | 1 |
| Expected @param tag for 'text'. | 1 |
| Expected @param tag for 'uri'. | 1 |
| Expected @param tag for 'utils'. | 1 |
| Expected @param tag for 'var'. | 1 |
| Expected @param tag for 'wagon'. | 1 |
| Expected @throws tag for 'AuthorizationException'. | 1 |
| Expected @throws tag for 'ChecksumVerificationException'. | 1 |
| Expected @throws tag for 'FileNotFoundException'. | 1 |
| Expected @throws tag for 'NoSuchGoalException'. | 1 |
| Expected @throws tag for 'ResourceDoesNotExistException'. | 1 |
| Expected @throws tag for 'TransferFailedException'. | 1 |
| Expected @throws tag for 'UnknownPluginException'. | 1 |
| Expression can be simplified. | 1 |
| Method length is 176 lines (max allowed is 175). | 1 |
| Method length is 232 lines (max allowed is 175). | 1 |
| Name 'PLUGIN_SCRIPT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'TEST_DOCUMENT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'TEST_DOCUMENT2' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPostcursor_DistantFullCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPostcursor_DistantPartialCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPostcursor_DistantPartialCycle_PieceWise' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPostcursor_NoCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPrecursor_DistantFullCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPrecursor_DistantPartialCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPrecursor_DistantPartialCycle_PieceWise' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPrecursor_LocalCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAddPrerequisite_NoCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAttainGoal_Bogus' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAttain_Linear' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAttain_Tree' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAttain_Tree_Overlap' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testAttain_Tree_PartiallyNotRequired' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testGetGoal_Bogus' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPercolate_Tree_Overlap' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPercolate_Tree_PartiallyNotRequired' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPostActionCallback_AddRemove' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPostAttain_Linear' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPostGoalCallback_AddRemove' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPostPercolate_Linear' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPostPercolate_Tree' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPostcursor_LocalCycle' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPreActionCallback_AddRemove' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testPreGoalCallback_AddRemove' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testRequiresAction_False' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Name 'testRequiresAction_True' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 1 |
| Unused @param tag for 'unpackedPluginDir'. | 1 |
| Unused @throws tag for 'IllegalArgumentException'. | 1 |
| Unused @throws tag for 'UnknownPluginException'. | 1 |
| Unused import - org.apache.commons.jelly.Tag. | 1 |
| Unused import - org.apache.maven.jelly.tags.maven.MavenTagLibrary. | 1 |
| Unused import - org.apache.maven.plugin.UnknownPluginException. | 1 |
| Variable 'cdelim' must be private and have accessor methods. | 1 |
| Variable 'cdelimChar' must be private and have accessor methods. | 1 |
| Variable 'cdelimSingleChar' must be private and have accessor methods. | 1 |
| Variable 'creturnDelims' must be private and have accessor methods. | 1 |
| Variable 'delimLast' must be private and have accessor methods. | 1 |
| Variable 'emacsMode' must be private and have accessor methods. | 1 |
| Variable 'lastToken' must be private and have accessor methods. | 1 |
| switch without "default" clause. | 1 |