< prev index next >

src/java.base/share/classes/jdk/internal/jrtfs/JrtPath.java

Print this page
rev 52979 : 8215281: Use String.isEmpty() when applicable in java.base
Reviewed-by: TBD

@@ -86,11 +86,11 @@
         }
     }
 
     @Override
     public final JrtPath getFileName() {
-        if (path.length() == 0)
+        if (path.isEmpty())
             return this;
         if (path.length() == 1 && path.charAt(0) == '/')
             return null;
         int off = path.lastIndexOf('/');
         if (off == -1)

@@ -208,11 +208,11 @@
     public final JrtPath relativize(Path other) {
         final JrtPath o = checkPath(other);
         if (o.equals(this)) {
             return new JrtPath(jrtfs, "", true);
         }
-        if (path.length() == 0) {
+        if (path.isEmpty()) {
             return o;
         }
         if (jrtfs != o.jrtfs || isAbsolute() != o.isAbsolute()) {
             throw new IllegalArgumentException(
                 "Incorrect filesystem or path: " + other);

@@ -260,20 +260,20 @@
         return jrtfs;
     }
 
     @Override
     public final boolean isAbsolute() {
-        return path.length() > 0 && path.charAt(0) == '/';
+        return !path.isEmpty() && path.charAt(0) == '/';
     }
 
     @Override
     public final JrtPath resolve(Path other) {
         final JrtPath o = checkPath(other);
-        if (this.path.length() == 0 || o.isAbsolute()) {
+        if (this.path.isEmpty() || o.isAbsolute()) {
             return o;
         }
-        if (o.path.length() == 0) {
+        if (o.path.isEmpty()) {
             return this;
         }
         StringBuilder sb = new StringBuilder(path.length() + o.path.length() + 1);
         sb.append(path);
         if (path.charAt(path.length() - 1) != '/')

@@ -299,11 +299,11 @@
         if (isAbsolute() != o.isAbsolute() || !tp.startsWith(op)) {
             return false;
         }
         int off = op.length();
         if (off == 0) {
-            return tp.length() == 0;
+            return tp.isEmpty();
         }
         // check match is on name boundary
         return tp.length() == off || tp.charAt(off) == '/' ||
                off == 0 || op.charAt(off - 1) == '/';
     }
< prev index next >