The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is
Effort is min
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 99 | 46 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| integral division result cast to double or float in net.miginfocom.demo.CallbackDemo$2.paintComponent(Graphics) | STYLE | ICAST_IDIV_CAST_TO_DOUBLE | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.miginfocom.demo.HiDPISimulator.GUI_BUF from instance method net.miginfocom.demo.HiDPISimulator$2.actionPerformed(ActionEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.miginfocom.demo.HiDPISimulator.GUI_BUF from instance method net.miginfocom.demo.HiDPISimulator$3.actionPerformed(ActionEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium | |
| Write to static field net.miginfocom.demo.HiDPISimulator.PAINT_GHOSTED from instance method net.miginfocom.demo.HiDPISimulator$3.actionPerformed(ActionEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.miginfocom.demo.HiDPISimulator.GUI_BUF from instance method net.miginfocom.demo.HiDPISimulator$4.itemStateChanged(ItemEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium | |
| Write to static field net.miginfocom.demo.HiDPISimulator.SCALE_LAYOUT from instance method net.miginfocom.demo.HiDPISimulator$4.itemStateChanged(ItemEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.miginfocom.demo.HiDPISimulator.GUI_BUF from instance method net.miginfocom.demo.HiDPISimulator$5.itemStateChanged(ItemEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium | |
| Write to static field net.miginfocom.demo.HiDPISimulator.SCALE_FONTS from instance method net.miginfocom.demo.HiDPISimulator$5.itemStateChanged(ItemEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium | |
| Write to static field net.miginfocom.demo.HiDPISimulator.SCALE_LAF from instance method net.miginfocom.demo.HiDPISimulator$5.itemStateChanged(ItemEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.miginfocom.demo.HiDPISimulator.GUI_BUF from instance method net.miginfocom.demo.HiDPISimulator$6.itemStateChanged(ItemEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new net.miginfocom.demo.SwingDemo() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | Medium | |
| Private method net.miginfocom.demo.SwingDemo.createPanel() is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.miginfocom.demo.SwtDemo.display from instance method new net.miginfocom.demo.SwtDemo() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.miginfocom.demo.SwtDemo$3.run() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in net.miginfocom.layout.CC.cell(int[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.endGroup(String[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.gap(String[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.grow(float[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.growPrio(int[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.shrink(float[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.shrinkPrio(int[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium | |
| Switch statement found in net.miginfocom.layout.CC.sizeGroup(String[]) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Check for oddness that won't work for negative numbers in net.miginfocom.layout.ConstraintParser.parseInsets(String, boolean) | STYLE | IM_BAD_CHECK_FOR_ODD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method new net.miginfocom.layout.Grid(ContainerWrapper, LC, AC, AC, Map, ArrayList) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.Grid.addDockingCell(int[], int, Grid$CompWrap) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.Grid.addToEndGroup(HashMap, String, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.Grid.getCell(int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.Grid.getRowGaps(DimConstraint[], int, boolean, boolean[]) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.Grid.setCell(int, int, Grid$Cell) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.Grid.wrap(int[], BoundSize) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| net.miginfocom.layout.Grid.getHeight() may expose internal representation by returning Grid.height | MALICIOUS_CODE | EI_EXPOSE_REP | Medium | |
| net.miginfocom.layout.Grid.getWidth() may expose internal representation by returning Grid.width | MALICIOUS_CODE | EI_EXPOSE_REP | Medium | |
| int value cast to float and then passed to Math.round in net.miginfocom.layout.Grid.layoutInOneDim(int, UnitValue, boolean, Float[]) | CORRECTNESS | ICAST_INT_CAST_TO_FLOAT_PASSED_TO_ROUND | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Primitive value is boxed and then immediately unboxed in net.miginfocom.layout.LayoutUtil.getBrokenBoundary(float, int, int) | PERFORMANCE | BX_BOXING_IMMEDIATELY_UNBOXED | Medium | |
| net.miginfocom.layout.LayoutUtil.setDesignTime(ContainerWrapper, boolean) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | Medium | |
| Method net.miginfocom.layout.LayoutUtil.calculateSerial(int[][], ResizeConstraint[], Float[], int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Method net.miginfocom.layout.LayoutUtil.getBrokenBoundary(float, int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Incorrect lazy initialization of static field net.miginfocom.layout.LayoutUtil.CR_MAP in net.miginfocom.layout.LayoutUtil.putCCString(Object, String) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | Medium | |
| Incorrect lazy initialization of static field net.miginfocom.layout.LayoutUtil.DT_MAP in net.miginfocom.layout.LayoutUtil.setDesignTime(ContainerWrapper, boolean) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.miginfocom.layout.LinkHandler.getValue(Object, String, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.miginfocom.layout.UnitValue.<static initializer>() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| net.miginfocom.layout.UnitValue.getSubUnits() may expose internal representation by returning UnitValue.subUnits | MALICIOUS_CODE | EI_EXPOSE_REP | Medium | |
| String is incompatible with expected argument type UnitConverter in net.miginfocom.layout.UnitValue.removeGlobalUnitConverter(String) | CORRECTNESS | GC_UNRELATED_TYPES | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.miginfocom.swing.SwingComponentWrapper.hasBaseline() invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | Medium | |
| Method net.miginfocom.swing.SwingComponentWrapper.getBaseline(int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | Medium | |
| Exception is caught when Exception is not thrown in net.miginfocom.swing.SwingComponentWrapper.getBaseline(int, int) | STYLE | REC_CATCH_EXCEPTION | Medium |