You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by ak...@apache.org on 2004/11/09 12:23:49 UTC

svn commit: rev 57018 - incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/message

Author: akarasulu
Date: Tue Nov  9 03:23:48 2004
New Revision: 57018

Modified:
   incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/message/LockableAttributeImpl.java
Log:
The snacc4j code which the test cases use to confirm correct operation of
snickers code does not keep track of attribute value order.  So the equality
test for an Attribute is not feasible for now.


Modified: incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/message/LockableAttributeImpl.java
==============================================================================
--- incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/message/LockableAttributeImpl.java	(original)
+++ incubator/directory/ldap/trunk/common/src/java/org/apache/ldap/common/message/LockableAttributeImpl.java	Tue Nov  9 03:23:48 2004
@@ -362,26 +362,26 @@
             return false;
         }
 
-        if ( attr.isOrdered() )
-        {
-            for ( int ii = 0; ii < attr.size(); ii++ )
-            {
-                try
-                {
-                    if ( ! list.get( ii).equals( attr.get( ii ) ) )
-                    {
-                        return false;
-                    }
-                }
-                catch ( NamingException e )
-                {
-                    e.printStackTrace();
-                    return false;
-                }
-            }
-        }
-        else
-        {
+//        if ( attr.isOrdered() )
+//        {
+//            for ( int ii = 0; ii < attr.size(); ii++ )
+//            {
+//                try
+//                {
+//                    if ( ! list.get( ii).equals( attr.get( ii ) ) )
+//                    {
+//                        return false;
+//                    }
+//                }
+//                catch ( NamingException e )
+//                {
+//                    e.printStackTrace();
+//                    return false;
+//                }
+//            }
+//        }
+//        else
+//        {
             for ( int ii = 0; ii < attr.size(); ii++ )
             {
                 try
@@ -397,7 +397,7 @@
                     return false;
                 }
             }
-        }
+//        }
 
         return true;
     }