RE: [9] RFR:8130360: Add tests to verify 3rd party security providers if they are in signed/unsigned modular JARs

2015-12-19 Thread Sibabrata Sahoo
Hi Mandy/Max, Here is the updated webrev with very minor change over previous: http://cr.openjdk.java.net/~asmotrak/siba/8130360/webrev.04/ Thanks Siba -Original Message- From: Sibabrata Sahoo Sent: Sunday, December 20, 2015 1:34 AM To: Mandy Chung; Weijun Wang Cc: Valerie Peng; jigsaw

RE: [9] RFR:8130360: Add tests to verify 3rd party security providers if they are in signed/unsigned modular JARs

2015-12-19 Thread Sibabrata Sahoo
Hi Mandy/Max, Please review the updated webrev: http://cr.openjdk.java.net/~asmotrak/siba/8130360/webrev.03/ "3rd party security providers" Test change includes: - Test are converted to use TestNG framework. - java/security/modules/JigsawSecurityUtils.java, renamed to "SecurityUtils.java". Prop

Re: Code Review Request, 8133070 Hot lock on BulkCipher.isAvailable

2015-12-19 Thread Xuelei Fan
new webrev: http://cr.openjdk.java.net/~xuelei/8133070/webrev.01/ Updates to webrev.00: Re-org the get methods of SSLContextImpl so as to avoid repeated initialization of instance fields. On 12/18/2015 11:35 PM, Sean Mullan wrote: > Here are a few other other comments on the code: > > SSLContext