RFR 8130648: JCK test api/java_security/AuthProvider/ProviderTests_login starts failing after JDK-7191662

Sean Mullan sean.mullan at oracle.com
Tue Sep 22 20:41:14 UTC 2015


On 09/18/2015 08:37 PM, Valerie Peng wrote:
> Sean,
>
> I have updated the webrev based on your suggestions and CCC has also
> been filed:
>
> Webrev: http://cr.openjdk.java.net/~valeriep/8130648/webrev.00/

SunPKCS11.java

110: can use diamond operator for anon classes now, ex: 
PrivilegedExceptionAction<>

116-118: wouldn't it be easier to do the following:

116             if (this.pHandler != null) {
117                 newOne.pHandler = this.pHandler;
118             }

then you don't need the catch block on lines 124-128.

--Sean





More information about the security-dev mailing list