Changeset 8357 in josm for trunk/src/org/openstreetmap/josm

Timestamp:

2015-05-15T23:49:31+02:00 (10 years ago)

Author:

Don-vip

Message:

fix some Findbugs violations

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/AlignInLineAction.java (modified) (1 diff)
• actions/JoinAreasAction.java (modified) (2 diffs)
• actions/RestartAction.java (modified) (1 diff)
• data/AutosaveTask.java (modified) (1 diff)
• data/Preferences.java (modified) (2 diffs)
• data/cache/JCSCacheManager.java (modified) (1 diff)
• data/coor/CachedLatLon.java (modified) (4 diffs)
• data/imagery/WmsCache.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/AlignInLineAction.java

@@ -337 +337 @@
      * Class that represent a line
      */
-    private class Line {
+    private static class Line {
 
         /**

trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

@@ -765 +765 @@
         List<WayInPolygon> result = new ArrayList<>();
 
-        //prepare prev and next maps
+        //prepare next map
         Map<Way, Way> nextWayMap = new HashMap<>();
-        Map<Way, Way> prevWayMap = new HashMap<>();
 
         for (int pos = 0; pos < parts.size(); pos ++) {
@@ -775 +774 @@
 
             nextWayMap.put(parts.get(pos), parts.get((pos + 1) % parts.size()));
-            prevWayMap.put(parts.get(pos), parts.get((pos + parts.size() - 1) % parts.size()));
         }
 

trunk/src/org/openstreetmap/josm/actions/RestartAction.java

@@ -114 +114 @@
                 String[] mainCommand = System.getProperty("sun.java.command").split(" ");
                 // look for a .jar in all chunks to support paths with spaces (fix #9077)
-                String jarPath = mainCommand[0];
-                for (int i = 1; i < mainCommand.length && !jarPath.endsWith(".jar"); i++) {
-                    jarPath += " " + mainCommand[i];
-                }
+                StringBuilder sb = new StringBuilder(mainCommand[0]);
+                for (int i = 1; i < mainCommand.length && !mainCommand[i-1].endsWith(".jar"); i++) {
+                    sb.append(" ").append(mainCommand[i]);
+                }
+                String jarPath = sb.toString();
                 // program main is a jar
                 if (jarPath.endsWith(".jar")) {

trunk/src/org/openstreetmap/josm/data/AutosaveTask.java

@@ -197 +197 @@
                 Main.warn(tr("Unable to delete old backup file {0}", oldFile.getAbsolutePath()));
             } else {
-                getPidFile(oldFile).delete();
+                File pidFile = getPidFile(oldFile);
+                if (!pidFile.delete()) {
+                    Main.warn(tr("Unable to delete old backup file {0}", pidFile.getAbsolutePath()));
+                }
             }
         }

trunk/src/org/openstreetmap/josm/data/Preferences.java

@@ -815 +815 @@
         File tmpFile = new File(prefFile + "_tmp");
         Utils.copyFile(tmpFile, prefFile);
-        tmpFile.delete();
+        if (!tmpFile.delete()) {
+            Main.warn(tr("Unable to delete temporary file {0}", tmpFile.getAbsolutePath()));
+        }
 
         setCorrectPermissions(prefFile);
@@ -822 +824 @@
 
     private void setCorrectPermissions(File file) {
-        file.setReadable(false, false);
-        file.setWritable(false, false);
-        file.setExecutable(false, false);
-        file.setReadable(true, true);
-        file.setWritable(true, true);
+        if (!file.setReadable(false, false) && Main.isDebugEnabled()) {
+            Main.debug(tr("Unable to set file non-readable {0}", file.getAbsolutePath()));
+        }
+        if (!file.setWritable(false, false) && Main.isDebugEnabled()) {
+            Main.debug(tr("Unable to set file non-writable {0}", file.getAbsolutePath()));
+        }
+        if (!file.setExecutable(false, false) && Main.isDebugEnabled()) {
+            Main.debug(tr("Unable to set file non-executable {0}", file.getAbsolutePath()));
+        }
+        if (!file.setReadable(true, true) && Main.isDebugEnabled()) {
+            Main.debug(tr("Unable to set file readable {0}", file.getAbsolutePath()));
+        }
+        if (!file.setWritable(true, true) && Main.isDebugEnabled()) {
+            Main.debug(tr("Unable to set file writable {0}", file.getAbsolutePath()));
+        }
     }
 

trunk/src/org/openstreetmap/josm/data/cache/JCSCacheManager.java

@@ -56 +56 @@
 
         File cacheDirLockPath = new File(cacheDir, ".lock");
-        if (!cacheDirLockPath.exists())
-            cacheDirLockPath.createNewFile();
+        if (!cacheDirLockPath.exists() && !cacheDirLockPath.createNewFile()) {
+            log.log(Level.WARNING, "Cannot create cache dir lock file");
+        }
         cacheDirLock = new FileOutputStream(cacheDirLockPath).getChannel().tryLock();
 

trunk/src/org/openstreetmap/josm/data/coor/CachedLatLon.java

@@ -9 +9 @@
  *
  * This class is convenient to use, but has relatively high memory costs.
- * It keeps a pointer to the last known projection in order to detect projection
- * changes.
+ * It keeps a pointer to the last known projection in order to detect projection changes.
  *
  * Node and WayPoint have another, optimized, cache for projected coordinates.
  */
 public class CachedLatLon extends LatLon {
+
+    private static final long serialVersionUID = 1L;
+
     private EastNorth eastNorth;
     private transient Projection proj;
 
+    /**
+     * Constructs a new {@code CachedLatLon}.
+     * @param lat latitude
+     * @param lon longitude
+     */
     public CachedLatLon(double lat, double lon) {
         super(lat, lon);
     }
 
+    /**
+     * Constructs a new {@code CachedLatLon}.
+     * @param coor lat/lon
+     */
     public CachedLatLon(LatLon coor) {
         super(coor.lat(), coor.lon());
@@ -27 +38 @@
     }
 
+    /**
+     * Constructs a new {@code CachedLatLon}.
+     * @param eastNorth easting/northing
+     */
     public CachedLatLon(EastNorth eastNorth) {
         super(Main.getProjection().eastNorth2latlon(eastNorth));
@@ -39 +54 @@
      */
     public final EastNorth getEastNorth() {
-        if(proj != Main.getProjection())
-        {
+        if (proj != Main.getProjection()) {
             proj = Main.getProjection();
             eastNorth = proj.latlon2eastNorth(this);
@@ -46 +60 @@
         return eastNorth;
     }
-    @Override public String toString() {
+
+    @Override
+    public String toString() {
         return "CachedLatLon[lat="+lat()+",lon="+lon()+"]";
     }
-
-    // Only for Node.get3892DebugInfo()
-    public Projection getProjection() {
-        return proj;
-    }
 }

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

@@ -112 +112 @@
     public WmsCache(String url, int tileSize) {
         File globalCacheDir = new File(cacheDirPath());
-        globalCacheDir.mkdirs();
+        if (!globalCacheDir.mkdirs()) {
+            Main.warn("Unable to create global cache directory: "+globalCacheDir.getAbsolutePath());
+        }
         cacheDir = new File(globalCacheDir, getCacheDirectory(url));
         cacheDir.mkdirs();
@@ -230 +232 @@
                 if (files != null) {
                     for (File file: files) {
-                        if (!referencedFiles.contains(file.getName())) {
-                            file.delete();
+                        if (!referencedFiles.contains(file.getName()) && !file.delete()) {
+                            Main.warn("Unable to delete file: "+file.getAbsolutePath());
                         }
                     }
@@ -530 +532 @@
         }
 
-        imageFile.getParentFile().mkdirs();
+        if (!imageFile.getParentFile().mkdirs()) {
+            Main.warn("Unable to create parent directory: "+imageFile.getParentFile().getAbsolutePath());
+        }
 
         if (img != null) {
@@ -555 +559 @@
                     }
                     totalFileSize -= size;
-                    file.delete();
+                    if (!file.delete()) {
+                        Main.warn("Unable to delete file: "+file.getAbsolutePath());
+                    }
                     it.remove();
                 }