Changeset 32912 in osm for applications/editors/josm/plugins/lakewalker/src

Timestamp:

2016-09-03T17:01:47+02:00 (8 years ago)

Author:

donvip

Message:

fix some error-prone warnings

Location:

applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker

Files:

• BooleanConfigurer.java (modified) (1 diff)
• DoubleConfigurer.java (modified) (1 diff)
• LakewalkerAction.java (modified) (4 diffs)
• LakewalkerPreferences.java (modified) (3 diffs)
• StringEnumConfigurer.java (modified) (1 diff)

applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/BooleanConfigurer.java

@@ -65 +65 @@
             box.setSelected(booleanValue().booleanValue());
             box.addItemListener(new java.awt.event.ItemListener() {
+                @Override
                 public void itemStateChanged(java.awt.event.ItemEvent e) {
                     setValue(box.isSelected());

applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/DoubleConfigurer.java

@@ -12 +12 @@
 
     public DoubleConfigurer(String key, String name) {
-        this(key, name, new Double(0));
+        this(key, name, 0d);
     }
 

applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerAction.java

@@ -58 +58 @@
     }
 
+    @Override
     public void actionPerformed(ActionEvent e) {
         if (Main.map == null || Main.map.mapView == null || active)
@@ -83 +84 @@
 
                 // sort files by date (most recent first)
-                Arrays.sort(wmsCache, new Comparator<File>() {
-                    public int compare(File f1, File f2) {
-                        return (int) (f2.lastModified()-f1.lastModified());
-                    }
-                });
+                Arrays.sort(wmsCache, Comparator.comparingLong(File::lastModified));
 
                 // delete aged or oversized, keep newest. Once size/age limit was reached delete all older files
@@ -296 +293 @@
     }
 
+    @Override
     public void mouseClicked(MouseEvent e) {
         if (active) {
@@ -305 +303 @@
     }
 
+    @Override
     public void mouseEntered(MouseEvent e) {
     }
 
+    @Override
     public void mouseExited(MouseEvent e) {
     }
 
+    @Override
     public void mousePressed(MouseEvent e) {
     }
 
+    @Override
     public void mouseReleased(MouseEvent e) {
     }

applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerPreferences.java

@@ -73 +73 @@
     }
 
+    @Override
     public void addGui(PreferenceTabbedPane gui) {
+        //CHECKSTYLE.OFF: LineLength
         maxSegsConfig.setToolTipText(tr("Maximum number of segments allowed in each generated way. Default 250."));
         maxNodesConfig.setToolTipText(tr("Maximum number of nodes to generate before bailing out (before simplifying lines). Default 50000."));
@@ -88 +90 @@
         maxCacheAgeConfig.setToolTipText(tr("Maximum age of each cached file in days. Default is 100"));
         sourceConfig.setToolTipText(tr("Data source text. Default is Landsat."));
+        //CHECKSTYLE.ON: LineLength
 
         /*String description =*/ tr("A plugin to trace water bodies on Landsat imagery.");
@@ -148 +151 @@
     * Save entered preference values on OK button
     */
+    @Override
     public boolean ok() {
         Main.pref.put(PREF_MAX_SEG, maxSegsConfig.getValueString());

applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/StringEnumConfigurer.java

@@ -50 +50 @@
             setValue(value);
             box.addActionListener(new ActionListener() {
+                @Override
                 public void actionPerformed(ActionEvent e) {
                     noUpdate = true;