Hi Emmanuel,

+1

SearchScope could be in 'org.apache.directory.shared.ldap.model.message'.
LdapURL could in 'org.apache.directory.shared.ldap.model.url'.

Shouldn't we also rename 'LdapURL' as 'LdapUrl' to match other class names 
(like Dn, Rdn, Oid, etc.)?

Regards,
Pierre-Arnaud 

On 30 mars 2011, at 02:31, Emmanuel Lecharny wrote:

> Hi guys,
> 
> while I was reviewing the Filter classes, I found a couple of classes in the 
> filter packages and I don't think they should be stored under this package :
> - SearchScope : an Enum used in SearchRequests
> - LdapURL : used mostly in referrals
> 
> Anyone has an idea about the best location for those 2 classes ? in 'message' 
> ?
> 
> -- 
> Regards,
> Cordialement,
> Emmanuel Lécharny
> www.iktek.com
> 

Reply via email to