Skip to content

Commit e94f191

Browse files
authored
Inspections - casts, java style array, simplify function closures (#187)
* remove redundant casts * use JAVA style array declaration * remove redundatnt continue as last statement simplify return command
1 parent d872546 commit e94f191

File tree

5 files changed

+5
-9
lines changed

5 files changed

+5
-9
lines changed

src/main/java/net/atomique/ksar/Main.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,8 @@ public static void main(String[] args) {
108108
} else {
109109
exit_error(resource.getString("INPUT_REQUIRE_ARG"));
110110
}
111-
continue;
112111
}
112+
113113
}
114114
}
115115

src/main/java/net/atomique/ksar/graph/Graph.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
import net.atomique.ksar.Config;
99
import net.atomique.ksar.GlobalOptions;
10-
import net.atomique.ksar.OSParser;
1110
import net.atomique.ksar.kSar;
1211
import net.atomique.ksar.ui.SortedTreeNode;
1312
import net.atomique.ksar.ui.TreeNodeInfo;
@@ -202,7 +201,7 @@ private boolean add_datapoint_plot(Second now, int col, String colheader, double
202201
// insert not possible
203202
// check if column can be update
204203
StatConfig statconfig =
205-
((OSParser) mysar.myparser).get_OSConfig().getStat(mysar.myparser.getCurrentStat());
204+
mysar.myparser.get_OSConfig().getStat(mysar.myparser.getCurrentStat());
206205
if (statconfig != null) {
207206
if (statconfig.canDuplicateTime()) {
208207
Number oldval = ((Stats.get(col))).getValue(now);

src/main/java/net/atomique/ksar/ui/GraphSelection.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ private void unselectallButtonActionPerformed(
121121
public void toggle_checkbox(JPanel panel, boolean checked) {
122122
Component[] list = panel.getComponents();
123123
for (int i = 0; i < list.length; i++) {
124-
Component tmp = (Component) list[i];
124+
Component tmp = list[i];
125125
if (tmp instanceof JPanel) {
126126
JPanel obj = (JPanel) tmp;
127127
toggle_checkbox(obj, checked);

src/main/java/net/atomique/ksar/ui/Preferences.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ public Preferences(java.awt.Frame parent) {
3131
}
3232

3333
private void load_landf() {
34-
UIManager.LookAndFeelInfo looks[] = UIManager.getInstalledLookAndFeels();
34+
UIManager.LookAndFeelInfo[] looks = UIManager.getInstalledLookAndFeels();
3535
for (int i = 0, n = looks.length; i < n; i++) {
3636
String tmp = looks[i].getName();
3737
UI_lanf_model.addElement(tmp);

src/main/java/net/atomique/ksar/xml/CnxHistory.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,7 @@ public boolean isValid() {
8181
if (hostname == null) {
8282
return false;
8383
}
84-
if (commandList.isEmpty()) {
85-
return false;
86-
}
87-
return true;
84+
return !commandList.isEmpty();
8885
}
8986

9087
public void dump() {

0 commit comments

Comments
 (0)