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 |
|---|---|---|---|
| 428 | 26 | 0 | 0 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.BuildMonitorImpl.panic(String) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 687 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.BuildMonitorImpl$7.actionPerformed(ActionEvent) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 643 |
| The class net.sourceforge.buildmonitor.BuildMonitorImpl$7 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should net.sourceforge.buildmonitor.BuildMonitorImpl$OpenURIInBrowserActionListener be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.BuildReport.getDate() may expose internal representation by returning BuildReport.date | MALICIOUS_CODE | EI_EXPOSE_REP | 189 |
| net.sourceforge.buildmonitor.BuildReport.BuildReport(String, Date, BuildReport$Status) may expose internal representation by storing an externally mutable object into BuildReport.date | MALICIOUS_CODE | EI_EXPOSE_REP2 | 175 |
| net.sourceforge.buildmonitor.BuildReport.setDate(Date) may expose internal representation by storing an externally mutable object into BuildReport.date | MALICIOUS_CODE | EI_EXPOSE_REP2 | 198 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.BuildReport$AgeComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.BuildReport$NameComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.dialogs.BambooPropertiesDialog.openBaseURLButtonActionPerformed(ActionEvent) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 467 |
| Method net.sourceforge.buildmonitor.dialogs.BambooPropertiesDialog.initComponents() invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 240 |
| Method net.sourceforge.buildmonitor.dialogs.BambooPropertiesDialog.updatePeriodFieldFocusLost(FocusEvent) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 494 |
| Exception is caught when Exception is not thrown in net.sourceforge.buildmonitor.dialogs.BambooPropertiesDialog.openBaseURLButtonActionPerformed(ActionEvent) | STYLE | REC_CATCH_EXCEPTION | 467 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to serverResponseIS in net.sourceforge.buildmonitor.monitors.BambooMonitor.getLatestBuildResults(String, String, String) | STYLE | DLS_DEAD_LOCAL_STORE | 681 |
| Dead store to returnedValue in net.sourceforge.buildmonitor.monitors.BambooMonitor.login(String, String, String) | STYLE | DLS_DEAD_LOCAL_STORE | 581 |
| net.sourceforge.buildmonitor.monitors.BambooMonitor.BambooMonitor(BuildMonitor) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 292 |
| Result of integer multiplication cast to long in net.sourceforge.buildmonitor.monitors.BambooMonitor.run() | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 332 |
| Method net.sourceforge.buildmonitor.monitors.BambooMonitor.run() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 324 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.monitors.BambooMonitor$BambooProperties.loadFromFile() may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 193 |
| net.sourceforge.buildmonitor.monitors.BambooMonitor$BambooProperties.saveToFile() may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 239 |
| Should net.sourceforge.buildmonitor.monitors.BambooMonitor$BambooProperties be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Result of integer multiplication cast to long in net.sourceforge.buildmonitor.monitors.CruiseControlRssMonitor.run() | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 124 |
| CruiseControlRssMonitor.lastBuildsStatus not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sourceforge.buildmonitor.utils.RssFeedItem.getPubDate() may expose internal representation by returning RssFeedItem.pubDate | MALICIOUS_CODE | EI_EXPOSE_REP | 73 |
| net.sourceforge.buildmonitor.utils.RssFeedItem.setPubDate(Date) may expose internal representation by storing an externally mutable object into RssFeedItem.pubDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 78 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should net.sourceforge.buildmonitor.utils.RssFeedReader$RssFeedContentHandler be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
