Re: RFR 8064890: SecureClassLoader should use a ConcurrentHashMap

2015-06-17 Thread Sean Mullan
On line 222, I would probably name the parameter "key" instead of "dummy" since that is what it is, even though it is not used. Looks good otherwise. Thanks, Sean On 06/17/2015 12:25 PM, Weijun Wang wrote: This fix is also a part for JEP 232 (Improve Secure Application Performance) [1]. webr

RFR 8064890: SecureClassLoader should use a ConcurrentHashMap

2015-06-17 Thread Weijun Wang
This fix is also a part for JEP 232 (Improve Secure Application Performance) [1]. webrev: http://cr.openjdk.java.net/~weijun/8064890/webrev.00/ bug: https://bugs.openjdk.java.net/browse/JDK-8064890 The fix uses a ConcurrentHashMap to avoid synchronization. Note: 1. ConcurrentHashMap does not