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 2007/02/13 19:23:33 UTC
svn commit: r507138 -
/directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/referral/ReferralLut.java
Author: elecharny
Date: Tue Feb 13 10:23:32 2007
New Revision: 507138
URL: http://svn.apache.org/viewvc?view=rev&rev=507138
Log:
Simple reformatting (adding some newlines, some curly braces)
Modified:
directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/referral/ReferralLut.java
Modified: directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/referral/ReferralLut.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/referral/ReferralLut.java?view=diff&rev=507138&r1=507137&r2=507138
==============================================================================
--- directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/referral/ReferralLut.java (original)
+++ directory/apacheds/branches/1.0/core/src/main/java/org/apache/directory/server/core/referral/ReferralLut.java Tue Feb 13 10:23:32 2007
@@ -84,8 +84,12 @@
public LdapDN getFarthestReferralAncestor( LdapDN dn )
{
if ( dn == null )
+ {
throw new IllegalArgumentException( "dn cannot be null" );
+ }
+
LdapDN farthest = new LdapDN();
+
for ( int ii = 0; ii < dn.size(); ii++ )
{
try
@@ -96,12 +100,14 @@
{
log.error( "Should never get this when moving names from a proper normalized name!", e );
}
+
// do not return dn if it is the farthest referral
- if ( isReferral( farthest ) && farthest.size() != dn.size() )
+ if ( isReferral( farthest ) && ( farthest.size() != dn.size() ) )
{
return farthest;
}
}
+
return null;
}