directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Emmanuel Lecharny (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DIRSHARED-97) Review Package Names
Date Mon, 14 Feb 2011 12:07:57 GMT

    [ https://issues.apache.org/jira/browse/DIRSHARED-97?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12994271#comment-12994271
] 

Emmanuel Lecharny commented on DIRSHARED-97:
--------------------------------------------

org.apache.directory.ldap.name.Dn sounds good. No need to repeat the 'shared' part as this
package is already in a jars with this name.

Btw, we should now switch to ldap-api instead of shared. Shared does not mean anything...

> Review Package Names
> --------------------
>
>                 Key: DIRSHARED-97
>                 URL: https://issues.apache.org/jira/browse/DIRSHARED-97
>             Project: Directory Shared
>          Issue Type: Task
>            Reporter: Stefan Seelmann
>             Fix For: 1.0.0-RC1
>
>
> The package names should be reviewed. Package names are not consistent and IMHO too long.
So here I just put some questions and ideas.
> Example from ldap-model:
> org.apache.directory.shared.ldap.model.name.Dn
> - Do we need the *shared* in package names? I think we can remove it, everything that
doesn't start with o.a.d.studio or o.a.d.server in the name is implicitly a 'shared' package.
> - Should we keep the *model* in package names? While I understand that during refactoring
it's easier to see the model packages, it isn't common in other APIs.
> The result could be: org.apache.directory.ldap.name.Dn
> Example from ldap-client-api:
> org.apache.directory.ldap.client.api.LdapConnection
> - Do we need the 'client.api' in package names? I think we can remove it, everything
with o.a.d.ldap in the name is implicitly the LDAP API.
> The result could be: org.apache.directory.ldap.LdapConnection

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message