Performance tweaks and code cleanup

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@5076 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Kevin Roast
2007-02-08 11:08:38 +00:00
parent 0249c6bbea
commit 268a821142
5 changed files with 6 additions and 10 deletions

View File

@@ -46,7 +46,4 @@ public class PermissionReferenceImpl extends AbstractPermissionReference
{
return qName;
}
}

View File

@@ -23,7 +23,7 @@ import org.alfresco.service.namespace.QName;
*
* @author andyh
*/
public class RequiredPermission extends PermissionReferenceImpl
public final class RequiredPermission extends PermissionReferenceImpl
{
public enum On {
PARENT, NODE, CHILDREN
@@ -49,5 +49,4 @@ public class RequiredPermission extends PermissionReferenceImpl
{
return on;
}
}

View File

@@ -34,7 +34,7 @@ import org.dom4j.Element;
*
* @author andyh
*/
public class PermissionGroup extends AbstractPermissionReference implements XMLModelInitialisable
public final class PermissionGroup extends AbstractPermissionReference implements XMLModelInitialisable
{
// XML Constants

View File

@@ -767,7 +767,7 @@ public class PermissionModel implements ModelDAO, InitializingBean
private Set<PermissionReference> getRequirementsForPermissionGroup(PermissionGroup target,
RequiredPermission.On on, QName qName, Set<QName> aspectQNames)
{
HashSet<PermissionReference> requiredPermissions = new HashSet<PermissionReference>(4, 1.0f);
HashSet<PermissionReference> requiredPermissions = new HashSet<PermissionReference>(8, 1.0f);
if (target == null)
{
return requiredPermissions;

View File

@@ -294,10 +294,10 @@ public final class QName implements QNamePattern, Serializable, Cloneable
}
if (object instanceof QName)
{
QName other = (QName) object;
QName other = (QName)object;
// namespaceURI and localname are not allowed to be null
return (this.namespaceURI.equals(other.namespaceURI) &&
this.localName.equals(other.localName));
return (this.localName.equals(other.localName) &&
this.namespaceURI.equals(other.namespaceURI));
}
else
{