Looks much better now:-)
-yan
On 09/15/2015 12:02 PM, Renjith Alexander wrote:
Hi Andrei,
Please see the updated version:
http://cr.openjdk.java.net/~pchopra/8132380/webrev.01/
Regards,
Renjith.
*From:*Andrei Eremeev
*Sent:* Friday, September 11, 2015 6:34 PM
*To:* Renjith Alexander; awt-dev@openjdk.java.net
*Subject:* Re: <AWT Dev> Review request for JDK-8132380: Do not create
LaF instance in javax/swing/plaf/windows/6921687/bug6921687.java
Hi Renjith,
Swing components should be created on the EDT to avoid concurrency problems.
So rollback the 55, 56, 61 and 62 lines.
Look at:
https://docs.oracle.com/javase/tutorial/uiswing/concurrency/index.html
Andrei
On 11.09.2015 13:16, Renjith Alexander wrote:
Hi all,
Please review the fix for the bug: JDK-8132380: Do not create LaF
instance in javax/swing/plaf/windows/6921687/bug6921687.java
BUG ID -*https://bugs.openjdk.java.net/browse/JDK-8132380 *
The webrev is:http://cr.openjdk.java.net/~pchopra/8132380/webrev.00/
<http://cr.openjdk.java.net/%7Epchopra/8132380/webrev.00/>
--
Regards,
Renjith Alexander
