PMD Results
The following document contains the results of PMD 4.2.2.
Files
net/sourceforge/basher/Average.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 136 |
net/sourceforge/basher/Basher.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 17 |
| Avoid unused imports such as 'java.util.Timer' | 18 |
| Avoid unused imports such as 'java.util.TimerTask' | 19 |
| Avoid unused imports such as 'bsh.EvalError' | 22 |
| Avoid unused imports such as 'bsh.Interpreter' | 23 |
net/sourceforge/basher/BasherContext.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 565 |
net/sourceforge/basher/BasherException.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 34 |
net/sourceforge/basher/TaskFailedException.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 36 |
net/sourceforge/basher/TaskManager.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 17 |
net/sourceforge/basher/TaskNotRunException.java
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 34 |
net/sourceforge/basher/annotations/BasherFollowers.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'net.sourceforge.basher.Phase' | 8 |
net/sourceforge/basher/impl/AbstractCollector.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'average'. | 213 |
net/sourceforge/basher/impl/AbstractFileCollector.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'net.sourceforge.basher.Task' | 24 |
| These nested if statements could be combined | 214 - 217 |
net/sourceforge/basher/impl/SchedulerImpl.java
| Violation | Line |
|---|---|
| Avoid using ThreadGroup; it is not thread safe | 261 |
| Avoid unused method parameters such as 'basherContext'. | 538 - 541 |
net/sourceforge/basher/impl/TaskTree.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'net.sourceforge.basher.Task' | 5 |
| Avoid unused private fields such as '_parents'. | 44 |
| Avoid empty if statements | 187 - 189 |
| Avoid unused method parameters such as 'lastPartTaskName'. | 209 - 227 |
net/sourceforge/basher/internal/TaskInvoker.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'net.sourceforge.basher.Task' | 17 |
net/sourceforge/basher/internal/impl/BasherBootHelper.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'net.sourceforge.basher.impl.TaskContribution' | 10 |
| Avoid unused imports such as 'org.apache.commons.logging.LogFactory' | 12 |
| Avoid unused local variables such as 'taskCreator'. | 52 |
| Avoid unused local variables such as 'taskManager'. | 53 |
| Avoid returning from a finally block | 111 |
| These nested if statements could be combined | 188 - 191 |
| These nested if statements could be combined | 196 - 199 |
net/sourceforge/basher/internal/impl/DecoratedTask.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 3 |
| Avoid unused imports such as 'java.util.ArrayList' | 4 |
| Avoid unused imports such as 'net.sourceforge.basher.Phase' | 7 |
net/sourceforge/basher/internal/impl/TaskConfigurationExtractorImpl.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.logging.impl.NoOpLog' | 7 |
| Avoid unused local variables such as 'annotation'. | 51 |
net/sourceforge/basher/internal/impl/TaskDecoratorImpl.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'net.sourceforge.basher.Phase' | 9 |
| Avoid unused imports such as 'net.sourceforge.basher.tasks.AbstractTask' | 12 |
| Avoid empty catch blocks | 99 - 102 |
net/sourceforge/basher/internal/impl/TaskInvokerImpl.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'task'. | 72 |