Re: AWT Dev [8] Review Request: 7172770 Default Toolkit implementation return null value for property awt.dynamicLayoutSupported

2013-10-25 Thread Artem Ananiev
On 10/24/2013 9:30 PM, Sergey Bylokhov wrote: Hi, Anthony. After a small considering with Artem I decided to reduce changes as we at zbb stage here is a new small version: http://cr.openjdk.java.net/~serb/7172770/webrev.01 Looks fine. Thanks, Artem On 24.10.2013 18:22, Anthony Petrov

AWT Dev [8] Review Request: 7172770 Default Toolkit implementation return null value for property awt.dynamicLayoutSupported

2013-10-24 Thread Sergey Bylokhov
Hello. Please review the fix for jdk 8. Implementation of dynamicLayoutSupported now aligned to the set/getdynamicLayout which always use default toolkit. Note that we have CR to revisit all this code JDK-8024948 Bug: https://bugs.openjdk.java.net/browse/JDK-7172770 Webrev can be found at:

Re: AWT Dev [8] Review Request: 7172770 Default Toolkit implementation return null value for property awt.dynamicLayoutSupported

2013-10-24 Thread Anthony Petrov
Hi Sergey, The fix looks good to me. -- best regards, Anthony On 10/24/2013 06:00 PM, Sergey Bylokhov wrote: Hello. Please review the fix for jdk 8. Implementation of dynamicLayoutSupported now aligned to the set/getdynamicLayout which always use default toolkit. Note that we have CR to

Re: AWT Dev [8] Review Request: 7172770 Default Toolkit implementation return null value for property awt.dynamicLayoutSupported

2013-10-24 Thread Sergey Bylokhov
Hi, Anthony. After a small considering with Artem I decided to reduce changes as we at zbb stage here is a new small version: http://cr.openjdk.java.net/~serb/7172770/webrev.01 On 24.10.2013 18:22, Anthony Petrov wrote: Hi Sergey, The fix looks good to me. -- best regards, Anthony On

Re: AWT Dev [8] Review Request: 7172770 Default Toolkit implementation return null value for property awt.dynamicLayoutSupported

2013-10-24 Thread Anthony Petrov
This looks even better. Thanks! -- best regards, Anthony On 10/24/2013 09:30 PM, Sergey Bylokhov wrote: Hi, Anthony. After a small considering with Artem I decided to reduce changes as we at zbb stage here is a new small version: http://cr.openjdk.java.net/~serb/7172770/webrev.01 On