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 |
---|---|---|---|---|
org.microcol.gui.MusicPlayer.playSound(String, int) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 41 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of org.microcol.gui.RightPanelPresenter.focusedTile, which is known to be non-null in org.microcol.gui.RightPanelPresenter.isItDifferentTile(Location) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 93 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.microcol.gui.Text.INSTANCE should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Medium | |
Write to static field org.microcol.gui.Text.INSTANCE from instance method new org.microcol.gui.Text(Text$Language) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 43 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.microcol.gui.panelview.GamePanelPresenter$PopUpDemo be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 95 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.microcol.gui.panelview.GamePanelView.paintOwnersFlag(Graphics2D, Point, Player) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 292 | Medium |