use isEmpty() instead of length check

vladak · vladak · commit 930dadfcfcb7 · 2025-09-01T20:07:06.000+02:00
diff --git a/opengrok-web/src/main/java/org/opengrok/web/PageConfig.java b/opengrok-web/src/main/java/org/opengrok/web/PageConfig.java
@@ -1220,7 +1220,7 @@ public boolean isDir() {
     }
 
     private static String trailingSlash(String path) {
-        return path.length() == 0 || path.charAt(path.length() - 1) != PATH_SEPARATOR
+        return path.isEmpty() || path.charAt(path.length() - 1) != PATH_SEPARATOR
                 ? PATH_SEPARATOR_STRING
                 : "";
     }

Original file line number	Diff line number	Diff line change
`@@ -1220,7 +1220,7 @@ public boolean isDir() {`
`1220`	`1220`	`}`
`1221`	`1221`
`1222`	`1222`	`private static String trailingSlash(String path) {`
`1223`		`- return path.length() == 0 \|\| path.charAt(path.length() - 1) != PATH_SEPARATOR`
	`1223`	`+ return path.isEmpty() \|\| path.charAt(path.length() - 1) != PATH_SEPARATOR`
`1224`	`1224`	`? PATH_SEPARATOR_STRING`
`1225`	`1225`	`: "";`
`1226`	`1226`	`}`