<AWT Dev> Review request for JDK-8132380: Do not create LaF instance in javax/swing/plaf/windows/6921687/bug6921687.java

[Renjith Alexander]

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 at 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: HYPERLINK "http://cr.openjdk.java.net/%7Epchopra/8132380/webrev.00/"http://cr.openjdk.java.net/~pchopra/8132380/webrev.00/
 
 
-- 
Regards,
Renjith Alexander

 

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20150915/0a4edb36/attachment-0001.html>