JDK 9 RFR for 8036841 : Reuse no-perms AccessControlContext object when performing isAuthorized check
This is a fix to avoid creating multiple no-perms ACC objects for access control checks. Since this is required in an uncommon scenario, I used the Initialization on demand holder pattern to only create it under those circumstances. webrev: http://cr.openjdk.java.net/~mullan/webrevs/8036841/webrev.01/ --Sean
Re: JDK 9 RFR for 8036841 : Reuse no-perms AccessControlContext object when performing isAuthorized check
Looks ok to me. Brad On 6/2/2014 11:15 AM, Sean Mullan wrote: This is a fix to avoid creating multiple no-perms ACC objects for access control checks. Since this is required in an uncommon scenario, I used the Initialization on demand holder pattern to only create it under those circumstances. webrev: http://cr.openjdk.java.net/~mullan/webrevs/8036841/webrev.01/ --Sean
RFR: 8037742 8041142: solaris sunpkcs11 config changes
Him I need review of two very simple config file changes to enable algorithms for Solaris PKCS#11 that were disabled due to fixed Solaris bugs or old SPARC hardware performance issues. http://cr.openjdk.java.net/~ascarpino/8037742/webrev/ http://cr.openjdk.java.net/~ascarpino/8041142/webrev/ thanks Tony
Re: RFR: 8037742 8041142: solaris sunpkcs11 config changes
Looks fine. Valerie On 06/02/14 12:34, Anthony Scarpino wrote: Him I need review of two very simple config file changes to enable algorithms for Solaris PKCS#11 that were disabled due to fixed Solaris bugs or old SPARC hardware performance issues. http://cr.openjdk.java.net/~ascarpino/8037742/webrev/ http://cr.openjdk.java.net/~ascarpino/8041142/webrev/ thanks Tony
Re: RFR: 8037742 8041142: solaris sunpkcs11 config changes
Thanks Tony On 06/02/2014 02:52 PM, Valerie (Yu-Ching) Peng wrote: Looks fine. Valerie On 06/02/14 12:34, Anthony Scarpino wrote: Him I need review of two very simple config file changes to enable algorithms for Solaris PKCS#11 that were disabled due to fixed Solaris bugs or old SPARC hardware performance issues. http://cr.openjdk.java.net/~ascarpino/8037742/webrev/ http://cr.openjdk.java.net/~ascarpino/8041142/webrev/ thanks Tony