(accumulo-access) branch main updated: Check for same object in AccessExpression.equals

[ctubbsii]

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 ce911c3  Check for same object in AccessExpression.equals
ce911c3 is described below

commit ce911c308c1f3c23eee5436476b527b4cb5db040
Author: Christopher Tubbs <ctubb...@apache.org>
AuthorDate: Tue Mar 10 14:39:30 2026 -0400

    Check for same object in AccessExpression.equals
    
    * Add missed code review suggestion from
      https://github.com/apache/accumulo-access/pull/102#discussion_r2911212085
    
    ---------
    
    Co-authored-by: Dave Marion <dlmar...@apache.org>
---
 .../src/main/java/org/apache/accumulo/access/AccessExpression.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/modules/core/src/main/java/org/apache/accumulo/access/AccessExpression.java 
b/modules/core/src/main/java/org/apache/accumulo/access/AccessExpression.java
index 4f90890..7c1d68e 100644
--- 
a/modules/core/src/main/java/org/apache/accumulo/access/AccessExpression.java
+++ 
b/modules/core/src/main/java/org/apache/accumulo/access/AccessExpression.java
@@ -53,7 +53,8 @@ public sealed abstract class AccessExpression implements 
Serializable
 
   @Override
   public boolean equals(Object o) {
-    return o instanceof AccessExpression a && Objects.equals(getExpression(), 
a.getExpression());
+    return this == o
+        || (o instanceof AccessExpression a && Objects.equals(getExpression(), 
a.getExpression()));
   }
 
   @Override