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.