Changeset 14624 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2018-12-31T19:36:59+01:00 (5 years ago)

Author:

Don-vip

Message:

fix various SonarQube issues

Location:

trunk/src/org/openstreetmap/josm/data

Files:

• gpx/GpxData.java (modified) (2 diffs)
• gpx/GpxImageEntry.java (modified) (4 diffs)
• imagery/TMSCachedTileLoader.java (modified) (2 diffs)
• projection/datum/NTV2Proj4DirGridShiftFileSource.java (modified) (2 diffs)
• validation/tests/UnconnectedWays.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java

@@ -489 +489 @@
                 GpxData d = new GpxData();
                 d.addTrack(trk);
-                return new GpxLayer(d, ensureUniqueName(attrs, counts)); })
+                return new GpxLayer(d, ensureUniqueName(attrs, counts));
+            })
             .forEachOrdered(layer -> MainApplication.getLayerManager().addLayer(layer));
     }
@@ -509 +510 @@
      */
     public synchronized int getTrackSegsCount() {
-        return privateTracks.stream().collect(Collectors.summingInt(t -> t.getSegments().size()));
+        return privateTracks.stream().mapToInt(t -> t.getSegments().size()).sum();
     }
 

trunk/src/org/openstreetmap/josm/data/gpx/GpxImageEntry.java

@@ -27 +27 @@
  * @since 14205 (extracted from gui.layer.geoimage.ImageEntry)
  */
-public class GpxImageEntry implements Comparable<GpxImageEntry>, Cloneable {
+public class GpxImageEntry implements Comparable<GpxImageEntry> {
     private File file;
     private Integer exifOrientation;
@@ -73 +73 @@
 
     /**
+     * Constructs a new {@code GpxImageEntry} from an existing instance.
+     * @param other existing instance
+     * @since 14624
+     */
+    public GpxImageEntry(GpxImageEntry other) {
+        file = other.file;
+        exifOrientation = other.exifOrientation;
+        exifCoor = other.exifCoor;
+        exifImgDir = other.exifImgDir;
+        exifTime = other.exifTime;
+        isNewGpsData = other.isNewGpsData;
+        exifGpsTime = other.exifGpsTime;
+        pos = other.pos;
+        speed = other.speed;
+        elevation = other.elevation;
+        gpsTime = other.gpsTime;
+        width = other.width;
+        height = other.height;
+        tmp = other.tmp;
+    }
+
+    /**
      * Constructs a new {@code GpxImageEntry}.
      * @param file Path to image file on disk
@@ -311 +333 @@
     public void setExifImgDir(Double exifDir) {
         this.exifImgDir = exifDir;
-    }
-
-    @Override
-    public GpxImageEntry clone() {
-        try {
-            return (GpxImageEntry) super.clone();
-        } catch (CloneNotSupportedException e) {
-            throw new IllegalStateException(e);
-        }
     }
 
@@ -370 +383 @@
      */
     public void createTmp() {
-        tmp = clone();
+        tmp = new GpxImageEntry(this);
         tmp.tmp = null;
     }

trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoader.java

@@ -78 +78 @@
      */
     public static ThreadPoolExecutor getNewThreadPoolExecutor(String nameFormat, int workers, int hostLimit) {
-        ThreadPoolExecutor executor = new ThreadPoolExecutor(
+        return new ThreadPoolExecutor(
                 workers, // keep core pool the same size as max, as we use unbounded queue so there will
                 workers, // be never more threads than corePoolSize
@@ -86 +86 @@
                 Utils.newThreadFactory(nameFormat, Thread.NORM_PRIORITY)
                 );
-        return executor;
     }
 

trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2Proj4DirGridShiftFileSource.java

@@ -10 +10 @@
 import java.util.Collections;
 import java.util.List;
+import java.util.stream.Collectors;
 
+import org.openstreetmap.josm.spi.preferences.Config;
 import org.openstreetmap.josm.tools.Logging;
 import org.openstreetmap.josm.tools.Platform;
@@ -74 +76 @@
     }
 
+    private static List<File> visit(String prefSuffix, String... defaults) {
+        return Config.getPref().getList("ntv2.proj4.grid.dir." + prefSuffix, Arrays.asList(defaults))
+                               .stream().map(File::new).collect(Collectors.toList());
+    }
+
     @Override
     public List<File> visitUnixoid() {
-        return Arrays.asList(new File("/usr/local/share/proj"), new File("/usr/share/proj"));
+        return visit("unix", "/usr/local/share/proj", "/usr/share/proj");
     }
 
     @Override
     public List<File> visitWindows() {
-        return Arrays.asList(new File("C:\\PROJ\\NAD"));
+        return visit("windows", "C:\\PROJ\\NAD");
     }
 

trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java

@@ -230 +230 @@
                     continue;
                 }
-                if (!s.highway && endnodesHighway.contains(en) && !s.w.concernsArea()) {
-                    map.put(en, s.w);
-                } else if (endnodes.contains(en) && !s.w.concernsArea()) {
+                if (((!s.highway && endnodesHighway.contains(en)) || endnodes.contains(en)) && !s.w.concernsArea()) {
                     map.put(en, s.w);
                 }