use isEmpty() instead of length check

Author: vladak

opengrok-web/src/main/webapp/list.jsp

@@ -146,8 +146,7 @@ document.pageReady.push(function() { pageReadyList();});
             if (!projects.contains(projectName)) {
                 projects.add(projectName);
                 // update cookie
-                cookieValue = cookieValue.length() == 0 ? projectName :
-                        projectName + ',' + cookieValue;
+                cookieValue = cookieValue.isEmpty() ? projectName : projectName + ',' + cookieValue;
                 Cookie cookie = new Cookie(PageConfig.OPEN_GROK_PROJECT, URLEncoder.encode(cookieValue, StandardCharsets.UTF_8));
                 // TODO hmmm, projects.jspf doesn't set a path
                 cookie.setPath(request.getContextPath() + '/');
@@ -210,7 +209,7 @@ document.pageReady.push(function() { pageReadyList();});
         }
 
         statistics.report(LOGGER, Level.FINE, "directory listing done", "dir.list.latency");
-    } else if (rev.length() != 0) {
+    } else if (!rev.isEmpty()) {
         // requesting a revision
         File xrefFile;
         if (cfg.isLatestRevision(rev) && (xrefFile = cfg.findDataFile()) != null) {

opengrok-web/src/main/webapp/mast.jsp

@@ -18,7 +18,7 @@ information: Portions Copyright [yyyy] [name of copyright owner]
 
 CDDL HEADER END
 
-Copyright (c) 2005, 2022, Oracle and/or its affiliates. All rights reserved.
+Copyright (c) 2005, 2025, Oracle and/or its affiliates. All rights reserved.
 Portions Copyright 2011 Jens Elkner.
 Portions Copyright (c) 2018, Chris Fraire <[email protected]>.
 
@@ -45,7 +45,7 @@ org.opengrok.indexer.web.Util"%>
     }
 
     String redir = cfg.canProcess();
-    if (redir == null || redir.length() > 0) {
+    if (redir == null || !redir.isEmpty()) {
         if (redir == null) {            
             response.sendError(HttpServletResponse.SC_NOT_FOUND);
         } else {