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 2006/11/07 22:29:17 UTC

svn commit: r472282 - /directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java

Author: akarasulu
Date: Tue Nov  7 13:29:16 2006
New Revision: 472282

URL: http://svn.apache.org/viewvc?view=rev&rev=472282
Log:
cleaning up unnecessary wrapping of an add entry operation in a composite operation

Modified:
    directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java

Modified: directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java
URL: http://svn.apache.org/viewvc/directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java?view=diff&rev=472282&r1=472281&r2=472282
==============================================================================
--- directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java (original)
+++ directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java Tue Nov  7 13:29:16 2006
@@ -82,8 +82,6 @@
         // Check an entry already exists.
         checkBeforeAdd( normalizedName );
 
-        CompositeOperation result = new CompositeOperation( csn );
-
         // Insert 'entryUUID' and 'entryDeleted'.
         entry = ( Attributes ) entry.clone();
         entry.remove( Constants.ENTRY_UUID );
@@ -96,8 +94,7 @@
         // MODIFY operation)
         entry.put( Constants.ENTRY_CSN, csn.toOctetString() );
 
-        result.add( new AddEntryOperation( csn, normalizedName, entry ) );
-        return result;
+        return new AddEntryOperation( csn, normalizedName, entry );
     }