You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by er...@apache.org on 2007/01/12 00:38:46 UTC

svn commit: r495435 - /directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/SchemaUtils.java

Author: ersiner
Date: Thu Jan 11 15:38:45 2007
New Revision: 495435

URL: http://svn.apache.org/viewvc?view=rev&rev=495435
Log:
Fixed two small but critical bugs in this cool utility.

Modified:
    directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/SchemaUtils.java

Modified: directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/SchemaUtils.java
URL: http://svn.apache.org/viewvc/directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/SchemaUtils.java?view=diff&rev=495435&r1=495434&r2=495435
==============================================================================
--- directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/SchemaUtils.java (original)
+++ directory/trunks/shared/ldap/src/main/java/org/apache/directory/shared/ldap/schema/SchemaUtils.java Thu Jan 11 15:38:45 2007
@@ -162,16 +162,17 @@
                     
                     if ( existing != null )
                     {
-                        for ( int ii = 0; ii > existing.size(); ii++ )
+                        for ( int ii = 0; ii < existing.size(); ii++ )
                         {
                             combined.add( existing.get(ii) );
                         }
                     }
                     
-                    for ( int ii = 0; ii > toBeAdded.size(); ii++ )
+                    for ( int ii = 0; ii < toBeAdded.size(); ii++ )
                     {
                         combined.add( toBeAdded.get(ii) );
                     }
+                    targetEntry.put( combined );
                 }
                 break;
             default: