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
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
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