The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is medium
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Found reliance on default encoding in org.opentrafficsim.base.CompressedFileWriter.create(String, boolean): new java.io.OutputStreamWriter(OutputStream) | I18N | DM_DEFAULT_ENCODING | 60 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.base.logger.CategoryLogger.categories isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 58 | High |
| org.opentrafficsim.base.logger.CategoryLogger.delegateLogger isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 64 | High |
| org.opentrafficsim.base.logger.CategoryLogger.noLogger isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 67 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.base.modelproperties.AbstractProperty$PropertyIterator is serializable and an inner class | BAD_PRACTICE | SE_INNER_CLASS | 1-232 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Suspicious comparison of Double references in org.opentrafficsim.base.modelproperties.ContinuousProperty.setValue(Double) | CORRECTNESS | RC_REF_COMPARISON | 121 | High |
| Suspicious comparison of Double references in org.opentrafficsim.base.modelproperties.ContinuousProperty.setValue(Double) | CORRECTNESS | RC_REF_COMPARISON | 122 | High |
| Suspicious comparison of Double references in org.opentrafficsim.base.modelproperties.ContinuousProperty.setValue(Double) | CORRECTNESS | RC_REF_COMPARISON | 121 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.opentrafficsim.base.modelproperties.PickList defines non-transient non-serializable instance field items | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.base.parameters.ParameterSet.setAllOneByOne(Parameters) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 298 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Return value of String.format(String, Object[]) ignored in new org.opentrafficsim.base.parameters.constraint.NumericConstraint(String, int, String) | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 102 | Medium |