Hi Sergey,
the fix looks good to me.
Thanks,
Alexander.
On 12/24/2014 03:57 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 9.
In the JDK-6727661 and JDK-6727662 the code : "= new
String("Pending")" was changed to = "Pending".
This object is used as a marker when we start initialization of
LazyValue.
"Pending" in the test and "Pending" in the UIDefaults are pointed to
the same object, so the loop in UIDefaults.getFromHashtable() will hang.
This flag was added in the JDK-4101618 and it does not seem the string
type was necessary, so I change it to the simple object, to prevent
similar regression in the future
Bug: https://bugs.openjdk.java.net/browse/JDK-7180976
Webrev can be found at:
http://cr.openjdk.java.net/~serb/7180976/webrev.00