From f782991655763d704279e38bccb4a0eabc03c646 Mon Sep 17 00:00:00 2001
From: Nicolas Capponi <nicolas.capponi@forgerock.com>
Date: Wed, 05 Nov 2014 16:30:15 +0000
Subject: [PATCH] OPENDJ-1581 CR-5092 Move Assertion.DEFAULT_COMPARATOR constant  to better suited ByteSequence interface

---
 opendj-core/src/main/java/org/forgerock/opendj/ldap/ByteSequence.java                    |    9 +++++++++
 opendj-core/src/main/java/org/forgerock/opendj/ldap/Assertion.java                       |   10 ----------
 opendj-core/src/main/java/org/forgerock/opendj/ldap/schema/AbstractMatchingRuleImpl.java |    2 +-
 3 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/opendj-core/src/main/java/org/forgerock/opendj/ldap/Assertion.java b/opendj-core/src/main/java/org/forgerock/opendj/ldap/Assertion.java
index cfd6747..1163c11 100644
--- a/opendj-core/src/main/java/org/forgerock/opendj/ldap/Assertion.java
+++ b/opendj-core/src/main/java/org/forgerock/opendj/ldap/Assertion.java
@@ -27,8 +27,6 @@
 package org.forgerock.opendj.ldap;
 
 
-import java.util.Comparator;
-
 import org.forgerock.opendj.ldap.spi.IndexQueryFactory;
 
 /**
@@ -51,14 +49,6 @@
         }
     };
 
-    /** A default ByteSequence comparator. */
-    public static final Comparator<ByteSequence> DEFAULT_COMPARATOR = new Comparator<ByteSequence>() {
-        @Override
-        public int compare(final ByteSequence o1, final ByteSequence o2) {
-            return o1.compareTo(o2);
-        }
-    };
-
     /**
      * Indicates whether the provided attribute value should be considered a
      * match for this assertion value according to the matching rule.
diff --git a/opendj-core/src/main/java/org/forgerock/opendj/ldap/ByteSequence.java b/opendj-core/src/main/java/org/forgerock/opendj/ldap/ByteSequence.java
index 62fc32a..308ec11 100755
--- a/opendj-core/src/main/java/org/forgerock/opendj/ldap/ByteSequence.java
+++ b/opendj-core/src/main/java/org/forgerock/opendj/ldap/ByteSequence.java
@@ -28,6 +28,7 @@
 
 import java.io.IOException;
 import java.io.OutputStream;
+import java.util.Comparator;
 
 /**
  * A {@code ByteSequence} is a readable sequence of byte values. This interface
@@ -35,6 +36,14 @@
  */
 public interface ByteSequence extends Comparable<ByteSequence> {
 
+    /** A default ByteSequence comparator. */
+    public static final Comparator<ByteSequence> DEFAULT_COMPARATOR = new Comparator<ByteSequence>() {
+        @Override
+        public int compare(final ByteSequence o1, final ByteSequence o2) {
+            return o1.compareTo(o2);
+        }
+    };
+
     /**
      * Returns a {@link ByteSequenceReader} which can be used to incrementally
      * read and decode data from this byte sequence.
diff --git a/opendj-core/src/main/java/org/forgerock/opendj/ldap/schema/AbstractMatchingRuleImpl.java b/opendj-core/src/main/java/org/forgerock/opendj/ldap/schema/AbstractMatchingRuleImpl.java
index da18e26..1e2aebb 100644
--- a/opendj-core/src/main/java/org/forgerock/opendj/ldap/schema/AbstractMatchingRuleImpl.java
+++ b/opendj-core/src/main/java/org/forgerock/opendj/ldap/schema/AbstractMatchingRuleImpl.java
@@ -106,7 +106,7 @@
 
     @Override
     public Comparator<ByteSequence> comparator(final Schema schema) {
-        return DEFAULT_COMPARATOR;
+        return ByteSequence.DEFAULT_COMPARATOR;
     }
 
     @Override

--
Gitblit v1.10.0