Looks fine.

-phil.

On 4/22/19 12:15 AM, Sergey Bylokhov wrote:
Hello.
Please review the fix for JDK 13.

Bug: https://bugs.openjdk.java.net/browse/JDK-8222799
Fix: http://cr.openjdk.java.net/~serb/8222799/webrev.00

The "java.beans.Introspector" uses "declaredMethodCache" which was used before JDK-4058433:
http://hg.openjdk.java.net/jdk/jdk/rev/ba387c302edd#l9.367
Note that "declaredMethodCache.put()" was removed, and now we never populate this cache. It is used only as a synchronization monitor in some methods.

In the fix this cache was removed, and synchronization was moved to the ThreadGroupContext class.



Reply via email to