The fix looks good for me.
Thanks, Alexandr. On 11/14/2013 5:50 PM, sergey malenkov wrote:
Hello, Could you please review the following fix: fix: http://cr.openjdk.java.net/~malenkov/8028054.8.0/ bug: https://bugs.openjdk.java.net/browse/JDK-8028054 There is an unsynchronized access to internal caches. I've synchronized both caches (for methods and constructors) . The 8023310 issue shows how is slow a simple synchronization. So I've created the Cache class that is similar to WeakIdentityMap. Later we could remove WeakIdentityMap which is a special case of Cache. Thanks, SAM
