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


Reply via email to