< 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,96 **** } } @Override public final JrtPath getFileName() { ! if (path.length() == 0) return this; if (path.length() == 1 && path.charAt(0) == '/') return null; int off = path.lastIndexOf('/'); if (off == -1) --- 86,96 ---- } } @Override public final JrtPath getFileName() { ! if (path.isEmpty()) return this; if (path.length() == 1 && path.charAt(0) == '/') return null; int off = path.lastIndexOf('/'); if (off == -1)
*** 208,218 **** public final JrtPath relativize(Path other) { final JrtPath o = checkPath(other); if (o.equals(this)) { return new JrtPath(jrtfs, "", true); } ! if (path.length() == 0) { return o; } if (jrtfs != o.jrtfs || isAbsolute() != o.isAbsolute()) { throw new IllegalArgumentException( "Incorrect filesystem or path: " + other); --- 208,218 ---- public final JrtPath relativize(Path other) { final JrtPath o = checkPath(other); if (o.equals(this)) { return new JrtPath(jrtfs, "", true); } ! if (path.isEmpty()) { return o; } if (jrtfs != o.jrtfs || isAbsolute() != o.isAbsolute()) { throw new IllegalArgumentException( "Incorrect filesystem or path: " + other);
*** 260,279 **** return jrtfs; } @Override public final boolean isAbsolute() { ! return path.length() > 0 && path.charAt(0) == '/'; } @Override public final JrtPath resolve(Path other) { final JrtPath o = checkPath(other); ! if (this.path.length() == 0 || o.isAbsolute()) { return o; } ! if (o.path.length() == 0) { return this; } StringBuilder sb = new StringBuilder(path.length() + o.path.length() + 1); sb.append(path); if (path.charAt(path.length() - 1) != '/') --- 260,279 ---- return jrtfs; } @Override public final boolean isAbsolute() { ! return !path.isEmpty() && path.charAt(0) == '/'; } @Override public final JrtPath resolve(Path other) { final JrtPath o = checkPath(other); ! if (this.path.isEmpty() || o.isAbsolute()) { return o; } ! 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,309 **** if (isAbsolute() != o.isAbsolute() || !tp.startsWith(op)) { return false; } int off = op.length(); if (off == 0) { ! return tp.length() == 0; } // check match is on name boundary return tp.length() == off || tp.charAt(off) == '/' || off == 0 || op.charAt(off - 1) == '/'; } --- 299,309 ---- if (isAbsolute() != o.isAbsolute() || !tp.startsWith(op)) { return false; } int off = op.length(); if (off == 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 >