[9] Review Request: 8072663: Remove the sun.security.acl package which is not used in the JDK

Alan Bateman Alan.Bateman at oracle.com
Wed Feb 18 07:03:06 UTC 2015


On 17/02/2015 20:59, Jason Uh wrote:
> Please review this fix, which removes the sun.security.acl package.
>
> webrev: http://cr.openjdk.java.net/~juh/8072663/00/
> jbs: http://bugs.openjdk.java.net/browse/JDK-8072663
>
> The sun.security.acl package is the default implementation of 
> java.security.acl but it's not used in JDK. The JCK tests for 
> java.security.acl only covers the three exception classes and there 
> are no JDK tests for the interfaces, implying that the spec does not 
> require the implementation of java.security.acl interfaces. This is 
> likely because java.security.acl package have been superseded by 
> classes in the java.security package since 1.2.
>
> Removing the sun.security.acl implementation will help offset the 
> increase in size to the base module that was caused by moving 
> java.security.acl from a separate dedicated module to java.base (see 
> http://bugs.openjdk.java.net/browse/JDK-8069551).
The changes looks good, assuming there is no way that the 
sun.security.acl classes will be used by anything else in the standard 
APi. I didn't realize there was a reference to the implementation 
classes in the java.security.acl javadoc.

-Alan


More information about the security-dev mailing list