altosuilib: Eliminate a couple java compiler warnings
authorKeith Packard <keithp@keithp.com>
Wed, 19 Feb 2020 22:01:15 +0000 (14:01 -0800)
committerKeith Packard <keithp@keithp.com>
Wed, 19 Feb 2020 22:01:15 +0000 (14:01 -0800)
Were using new Integer(int), when just passing the int would
'auto-box' as necessary.

Signed-off-by: Keith Packard <keithp@keithp.com>
altosuilib/AltosUIConfigure.java
altosuilib/AltosUIEnable.java

index e61b5d5205d96d15dc5816c0c6663188fa437d35..8f68793e2ec8b4c8dcdb8d0f845ea03b48e10f10 100644 (file)
@@ -240,12 +240,12 @@ public class AltosUIConfigure
 
                                                AltosUIPreferences.set_map_cache(size);
                                        } catch (ClassCastException ce) {
-                                               map_cache.setSelectedItem(new Integer(AltosUIPreferences.map_cache()));
+                                               map_cache.setSelectedItem(AltosUIPreferences.map_cache());
                                        }
                                }
                        });
 
-               map_cache.setSelectedItem (new Integer(AltosUIPreferences.map_cache()));
+               map_cache.setSelectedItem (AltosUIPreferences.map_cache());
                pane.add(map_cache, constraints(1, 2, GridBagConstraints.BOTH));
                row++;
        }
index ef9a692a62a47ce07339378d6dc92e6d64e53b19..0cb562d22c40ecf72903d3a79c7da6e91f1c7450 100644 (file)
@@ -175,15 +175,14 @@ public class AltosUIEnable extends Container implements ChangeListener {
                add(line_width_label, c);
 
                line_width = new JSpinner();
-               line_width.setValue(new Integer(1));
+               line_width.setValue(1);
                line_width.addChangeListener(new ChangeListener() {
                                public void stateChanged(ChangeEvent e) {
                                        int w = (Integer) line_width.getValue();
                                        if (w < 1) {
                                                w = 1;
-                                               line_width.setValue(new Integer(w));
+                                               line_width.setValue(w);
                                        }
-                                       System.out.printf("line width set to %d\n", w);
                                        set_line_width(w);
                                }
                        });
@@ -205,7 +204,7 @@ public class AltosUIEnable extends Container implements ChangeListener {
                speed_filter = new JSlider(JSlider.HORIZONTAL, 0, 10000, (int) (filter_listener.speed_filter() * 1000.0));
                Hashtable<Integer,JLabel> label_table = new Hashtable<Integer,JLabel>();
                for (int i = 0; i <= 10000; i += 5000) {
-                       label_table.put(new Integer(i), new JLabel(String.format("%d", i)));
+                       label_table.put(i, new JLabel(String.format("%d", i)));
                }
                speed_filter.setPaintTicks(true);
                speed_filter.setMajorTickSpacing(1000);