Thanks for review.
Andrei
On 06/23/2014 05:52 PM, Petr Pchelko wrote:
Hello, Andrei.
Looks good.
With best regards. Petr.
On 23 июня 2014 г., at 17:33, andrei.eremeev <andrei.erem...@oracle.com> wrote:
Ok, fixed.
The fix:
http://cr.openjdk.java.net/~yan/7112454/webrev.diff.05/
Test moved to open:
http://cr.openjdk.java.net/~yan/7112454/webrev.05/
Thanks,
Andrei
On 06/23/2014 04:39 PM, Petr Pchelko wrote:
Hello, Andrei.
HTML file still lacks copyright header.
With best regards. Petr.
On 23 июня 2014 г., at 15:22, andrei.eremeev <andrei.erem...@oracle.com> wrote:
Fixed Petr's remarks.
The fix:
http://cr.openjdk.java.net/~yan/7112454/webrev.diff.04/
Test moved to open:
http://cr.openjdk.java.net/~yan/7112454/webrev.04/
Andrei
On 01/21/2014 04:41 PM, Petr Pchelko wrote:
Hello, Andrei.
Please update the copyright header, it should say 2014. And the header is
missing in an html file.
As you’ve added Util you could replace the dragMouse method with Util.drag and
remove the method.
With best regards. Petr.
21 янв. 2014 г., в 4:14 после полудня, andrei.eremeev
<andrei.erem...@oracle.com> написал(а):
Hi, AWT team.
Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-7112454
The fix is available at:
http://cr.openjdk.java.net/~yan/7112454/webrev.diff.03/
Test moved to open:
http://cr.openjdk.java.net/~yan/7112454/webrev.03/