The following document contains the results of PMD 3.7.
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 7 - 49 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 112 - 115 |
| Violation | Line |
|---|---|
| Use explicit scoping instead of the default package private level | 20 |
| Use explicit scoping instead of the default package private level | 22 |
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 33 |
| Violation | Line |
|---|---|
| Perhaps 'frameBufferColumn' could be replaced by a local variable. | 35 |
| Perhaps 'searchDialog' could be replaced by a local variable. | 41 |
| A method should have only one exit point, and that should be the last statement in the method | 59 |
| A method should have only one exit point, and that should be the last statement in the method | 61 |
| A method should have only one exit point, and that should be the last statement in the method | 79 |
| A method should have only one exit point, and that should be the last statement in the method | 93 |
| Assigning an Object to null is a code smell. Consider refactoring. | 213 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 34 - 43 |
| These nested if statements could be combined | 117 - 118 |
| Violation | Line |
|---|---|
| Perhaps 'menuBox' could be replaced by a local variable. | 36 |
| Perhaps 'entries' could be replaced by a local variable. | 63 |
| Perhaps 'toggleEntries' could be replaced by a local variable. | 80 |
| Perhaps 'toggleShowEntries' could be replaced by a local variable. | 84 |
| Perhaps 'uiInfo' could be replaced by a local variable. | 110 |