On Wed, Mar 30, 2011 at 12:20 PM, Pierre-Arnaud Marcelot <pa@marcelot.net> wrote:
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


My same thoughts. LdapUrl should go into message pkg too though - why have an extra package with just one or two classes?

Regards,
Alex