On 8/17/17 8:16 PM, Weijun Wang wrote:
Hi Sean

Change looks fine.

And I found another 4 references in comments in 
jdk/src/java.base/share/classes/javax/security/auth/Policy.java.

Good catch. I changed SolarisPrincipal and SolarisNumericUserPrincipal to UnixPrincipal and UnixNumericUserPrincipal, respectively.

Updated webrev: http://cr.openjdk.java.net/~mullan/webrevs/8159544/jdk.webrev.01/

BTW, do we have a test to show what lines in various Resources.java files are 
used where and if one is useless?

I don't know. I have copied Naoto to see if he knows.

--Sean


Thanks
Max

On Aug 18, 2017, at 3:08 AM, Sean Mullan <sean.mul...@oracle.com> wrote:

Please review this JDK 10 change to remove the deprecated classes in 
com.sun.security.auth.** that have been previously marked with forRemoval=true 
in JDK 9.

webrev: http://cr.openjdk.java.net/~mullan/webrevs/8159544/

I have also copied Jan for reviewing a change in langtools, and also build-dev 
for a change to one of the JDK Makefiles.

Thanks,
Sean

Reply via email to