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