This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-access.git


The following commit(s) were added to refs/heads/main by this push:
     new 884298c  Replace an additional call to term.length()
884298c is described below

commit 884298c88c966aa6368db6f2398e63f7f666d064
Author: Christopher Tubbs <[email protected]>
AuthorDate: Tue Mar 10 17:28:34 2026 -0400

    Replace an additional call to term.length()
---
 .../java/org/apache/accumulo/access/impl/AccessExpressionImpl.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
 
b/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
index 6f4e5bf..42fd3dc 100644
--- 
a/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
+++ 
b/modules/core/src/main/java/org/apache/accumulo/access/impl/AccessExpressionImpl.java
@@ -77,9 +77,9 @@ public final class AccessExpressionImpl extends 
AccessExpression {
   }
 
   public static CharSequence unquote(CharSequence term) {
-    int len = term.length();
+    final int len = term.length();
     if (len >= 2 && term.charAt(0) == '"' && term.charAt(len - 1) == '"') {
-      term = len == 2 ? "" : AccessEvaluatorImpl.unescape(term.subSequence(1, 
term.length() - 1));
+      term = len == 2 ? "" : AccessEvaluatorImpl.unescape(term.subSequence(1, 
len - 1));
     }
     if (term.isEmpty()) {
       throw new IllegalArgumentException("Empty strings are not legal 
authorizations.");

Reply via email to