RFR 8064890: SecureClassLoader should use a ConcurrentHashMap

Sean Mullan sean.mullan at oracle.com
Wed Jun 17 20:08:53 UTC 2015


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].
>
> 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 accept null keys.
>
> 2. Lambda cannot be used at such an early stage.
>
> Noreg-cleanup.
>
> Thanks
> Max
>
>
> [1] http://openjdk.java.net/jeps/232



More information about the security-dev mailing list