Changeset 11893 in josm for trunk/src/org/openstreetmap/josm/gui/io/LayerNameAndFilePathTableCell.java

Timestamp:

2017-04-13T01:08:58+02:00 (9 years ago)

Author:

Don-vip

Message:

sonar - squid:S1126 - Return of boolean expressions should not be wrapped into an "if-then-else" statement

File:

• trunk/src/org/openstreetmap/josm/gui/io/LayerNameAndFilePathTableCell.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/io/LayerNameAndFilePathTableCell.java

@@ -124 +124 @@
 
     private static boolean canWrite(File f) {
-        if (f == null) return false;
-        if (f.isDirectory()) return false;
+        if (f == null || f.isDirectory()) return false;
         if (f.exists() && f.canWrite()) return true;
-        if (!f.exists() && f.getParentFile() != null && f.getParentFile().canWrite())
-            return true;
-        return false;
+        return !f.exists() && f.getParentFile() != null && f.getParentFile().canWrite();
     }