Re: RFR 9: 8169416: SSLSessionImpl finalize overhead

2016-11-22 Thread Xuelei Fan
Looks fine to me. Thanks, Xuelei > On 23 Nov 2016, at 5:41 AM, Roger Riggs wrote: > > Adding security-dev... > > Please review this change to remove an ineffective finalizer for SSLSessions. > The finalizer removes bindings from the SSLSession of a table that is also > freed when the SSLSessio

Re: RFR 9: 8169416: SSLSessionImpl finalize overhead

2016-11-22 Thread Roger Riggs
Adding security-dev... Please review this change to remove an ineffective finalizer for SSLSessions. The finalizer removes bindings from the SSLSession of a table that is also freed when the SSLSession is being freed. There is no point in removing them explicitly. It only delays freeing memory