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.java110: 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
