You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2012/04/27 22:24:02 UTC
svn commit: r1331575 -
/directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/shared/ldap/model/entry/DefaultAttribute.java
Author: elecharny
Date: Fri Apr 27 20:24:01 2012
New Revision: 1331575
URL: http://svn.apache.org/viewvc?rev=1331575&view=rev
Log:
Save some CPU by avoiding some useless computaton done on a known upId
Modified:
directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/shared/ldap/model/entry/DefaultAttribute.java
Modified: directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/shared/ldap/model/entry/DefaultAttribute.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/shared/ldap/model/entry/DefaultAttribute.java?rev=1331575&r1=1331574&r2=1331575&view=diff
==============================================================================
--- directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/shared/ldap/model/entry/DefaultAttribute.java (original)
+++ directory/shared/trunk/ldap/model/src/main/java/org/apache/directory/shared/ldap/model/entry/DefaultAttribute.java Fri Apr 27 20:24:01 2012
@@ -1923,8 +1923,16 @@ public class DefaultAttribute implements
try
{
DefaultAttribute attribute = ( DefaultAttribute ) super.clone();
- attribute.setUpId( upId );
+ if ( this.attributeType != null )
+ {
+ attribute.id = attributeType.getOid();
+ attribute.attributeType = attributeType;
+ }
+
+ // Compute the hashCode
+ rehash();
+
attribute.values = new LinkedHashSet<Value<?>>( values.size() );
for ( Value<?> value : values )