The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is medium
Effort is min
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 140 | 41 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to x in javax.microedition.lcdui.ChoiceGroup.getItemIndexAt(int, int) | STYLE | DLS_DEAD_LOCAL_STORE | 543 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of javax.microedition.lcdui.DateCanvas.day; locked 80% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 278 | Medium |
| Inconsistent synchronization of javax.microedition.lcdui.DateCanvas.month; locked 78% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 277 | Medium |
| Inconsistent synchronization of javax.microedition.lcdui.DateCanvas.selected; locked 72% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 335 | Medium |
| Inconsistent synchronization of javax.microedition.lcdui.DateCanvas.year; locked 76% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 276 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| javax.microedition.lcdui.DateField.getDate() may expose internal representation by returning DateField.date | MALICIOUS_CODE | EI_EXPOSE_REP | 111 | Medium |
| javax.microedition.lcdui.DateField.setDate(Date) may expose internal representation by storing an externally mutable object into DateField.date | MALICIOUS_CODE | EI_EXPOSE_REP2 | 120 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field javax.microedition.lcdui.Alert.nextDisplayable from instance method javax.microedition.lcdui.Display.setCurrent(Alert, Displayable) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 587 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should javax.microedition.lcdui.Display$AlertTimeout be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 412-421 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method javax.microedition.lcdui.Font.getFont(int, int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 97 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of javax.microedition.lcdui.StringComponent.text; locked 84% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 128 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of javax.microedition.lcdui.TimeCanvas.hours; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 497 | Medium |
| Inconsistent synchronization of javax.microedition.lcdui.TimeCanvas.minutes; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 498 | Medium |
| Inconsistent synchronization of javax.microedition.lcdui.TimeCanvas.selected; locked 80% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 549 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to img in javax.microedition.lcdui.game.Sprite.collidesWith(Object, int, int) | STYLE | DLS_DEAD_LOCAL_STORE | 645 | Medium |
| javax.microedition.lcdui.game.Sprite.setFrameSequence(int[]) may expose internal representation by storing an externally mutable object into Sprite.sequence | MALICIOUS_CODE | EI_EXPOSE_REP2 | 260 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to cH in javax.microedition.lcdui.game.TiledLayer.paint(Graphics) | STYLE | DLS_DEAD_LOCAL_STORE | 231 | Medium |
| Dead store to cW in javax.microedition.lcdui.game.TiledLayer.paint(Graphics) | STYLE | DLS_DEAD_LOCAL_STORE | 230 | Medium |
| Dead store to cX in javax.microedition.lcdui.game.TiledLayer.paint(Graphics) | STYLE | DLS_DEAD_LOCAL_STORE | 228 | Medium |
| Dead store to cY in javax.microedition.lcdui.game.TiledLayer.paint(Graphics) | STYLE | DLS_DEAD_LOCAL_STORE | 229 | Medium |
| Dead store to imgRows in javax.microedition.lcdui.game.TiledLayer.paint(Graphics) | STYLE | DLS_DEAD_LOCAL_STORE | 256 | Medium |
| Inconsistent synchronization of javax.microedition.lcdui.game.TiledLayer.img; locked 90% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 645 | Medium |
| Self assignment of field TiledLayer.img in javax.microedition.lcdui.game.TiledLayer.setStaticTileSet(Image, int, int) | CORRECTNESS | SA_FIELD_SELF_ASSIGNMENT | 154 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.microemu.app.launcher.Launcher.midletEntries isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 49 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.microemu.device.InputMethodEvent.type | PERFORMANCE | URF_UNREAD_FIELD | 41 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of org.microemu.device.ui.EventDispatcher.lastPaintEventTime; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 94 | Medium |
| org.microemu.device.ui.EventDispatcher.maxFps isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 33 | Medium |
| Naked notify in org.microemu.device.ui.EventDispatcher.cancel() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 111 | Medium |
| Wait not in loop in org.microemu.device.ui.EventDispatcher.serviceRepaints() | MT_CORRECTNESS | WA_NOT_IN_LOOP | 157 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.microemu.util.RecordEnumerationImpl$EnumerationRecord be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 268-271 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.microemu.util.RecordStoreImpl.addRecord(byte[], int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 337 | Medium |
| Method org.microemu.util.RecordStoreImpl.deleteRecord(int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 362 | Medium |
| Method org.microemu.util.RecordStoreImpl.getRecord(int, byte[], int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 402 | Medium |
| Method org.microemu.util.RecordStoreImpl.getRecordSize(int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 382 | Medium |
| Method org.microemu.util.RecordStoreImpl.readRecord(DataInputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 126 | Medium |
| Method org.microemu.util.RecordStoreImpl.setRecord(int, byte[], int, int) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 447 | Medium |
| Inconsistent synchronization of org.microemu.util.RecordStoreImpl.lastModified; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 139 | Medium |
| Inconsistent synchronization of org.microemu.util.RecordStoreImpl.lastRecordId; locked 60% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 120 | Medium |
| Inconsistent synchronization of org.microemu.util.RecordStoreImpl.size; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 143 | Medium |
| Inconsistent synchronization of org.microemu.util.RecordStoreImpl.version; locked 77% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 140 | Medium |
| org.microemu.util.RecordStoreImpl.readRecord(DataInputStream) ignores result of java.io.DataInputStream.read(byte[], int, int) | BAD_PRACTICE | RR_NOT_CHECKED | 125 | Medium |