The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 64 | 6 | 0 | 0 |
| Class | Bugs |
|---|---|
| org.andromda.templateengines.velocity.VelocityTemplateEngine | 4 |
| org.andromda.templateengines.velocity.VelocityTemplateEngine$VelocityLoggingReceiver | 2 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.andromda.templateengines.velocity.VelocityTemplateEngine.logger should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 48 |
| Redundant nullcheck of templateObjects, which is known to be non-null in org.andromda.templateengines.velocity.VelocityTemplateEngine.loadVelocityContext(Map) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 254 |
| VelocityTemplateEngine.velocityContext not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
| Method org.andromda.templateengines.velocity.VelocityTemplateEngine.loadVelocityContext(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 259 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.andromda.templateengines.velocity.VelocityTemplateEngine$VelocityLoggingReceiver.logVelocityMessage(int, String) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 422-423 |
| Method org.andromda.templateengines.velocity.VelocityTemplateEngine$VelocityLoggingReceiver.logVelocityMessage(int, String) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 428-429 |
