Both looks fine to me. Thanks, Xuelei
On 7/27/2012 9:13 AM, Valerie (Yu-Ching) Peng wrote: > Thanks Max and Xuelei for your reviews! > I've updated the webrevs for 7107613, and 7107616: > > http://cr.openjdk.java.net/~valeriep/7107613/webrev.01 > http://cr.openjdk.java.net/~valeriep/7107616/webrev.01 > > Please let me know if you have more comments. > Thanks, > Valerie > > On 07/25/12 17:21, Valerie (Yu-Ching) Peng wrote: >> Hi, Max or Xuelei, >> >> Either of you have cycles to review the following 3 perf related fixes? >> 7107613: scalability bloker in javax.crypto.CryptoPermissions >> 7107616: scalability bloker in javax.crypto.JceSecurityManager >> 7185471: Avoid key expansion when AES cipher is re-init w/ the same key >> >> Webrevs are at: >> http://cr.openjdk.java.net/~valeriep/7107613 >> http://cr.openjdk.java.net/~valeriep/7107616 >> http://cr.openjdk.java.net/~valeriep/7185471 >> >> The changes are for JDK 8. May be backported to 7u later if necessary, >> Thanks, >> Valerie >
