Re: [9] Review request for 8033699: Incorrect radio button behavior

2014-09-10 Thread Vivi An
Hello Alexander, Thanks for nice review, new patch is available as below: webrev: http://cr.openjdk.java.net/~van/8033699/webrev.01/ Regards, ~ Vivi On 9/8/2014 5:43 AM, Alexander Scherbatiy wrote: On 9/4/2014 10:56 PM, Vivi An wrote: Hello, Please review fix for JDK-8033699. Changes made

Re: [9] Review request for 8048110: Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow

2014-09-10 Thread Alexander Potochkin
looks good Thanks alexp On 9/9/2014 1:53 PM, dmitry markov wrote: Hi Alexandr, I have added updateAfterChange() method as you suggested. New version is located at http://cr.openjdk.java.net/~dmarkov/8048110/jdk9/webrev.03/ Thanks, Dmitry On 08/09/2014 14:44, Alexander Scherbatiy wrote: On

Re: [9] Review Request: 8055326 Fix typos in client-related packages

2014-09-10 Thread Sergey Bylokhov
Hi, Phil. It seems both changes are unnecessary: http://cr.openjdk.java.net/~serb/8055326/webrev.01 On 21.08.2014 22:04, Phil Race wrote: Was the additional spce on the 2nd line intended here ? --- old/src/java.desktop/share/classes/java/awt/ContainerOrderFocusTraversalPolicy.java 2014-08-21