Looks fine.
On 29.09.2014 16:11, Alexander Potochkin wrote:
Hello Dmitry
The changes look good!
Thanks
alexp
On 9/10/2014 4:25 PM, dmitry markov wrote:
Thank you, Sean!
Anthony, Sergey,
Could you review this, please?
Thanks,
Dmitry
On 10/09/2014 16:19, Seán Coffey wrote:
Approved but subject to code review.
regards,
Sean.
On 10/09/14 13:16, dmitry markov wrote:
Hello,
Could you approve the back-port of the fix for 8028484 to jdk7u-dev
(7u80), please?
The original fix and the back-port are slightly different. There
are trivial changes in the regression test (lambda was replaced by
anonymous inner class). The author and the reviewers of the
original fix are in the CC list.
The bug:
https://bugs.openjdk.java.net/browse/JDK-8028484
The webrev:
http://cr.openjdk.java.net/~dmarkov/8028484/open/webrev.00/ - add
to open repo
http://cr.openjdk.java.net/~dmarkov/8028484/closed/webrev.00/ -
diff with the previous version of the test
The review thread:
http://mail.openjdk.java.net/pipermail/awt-dev/2013-December/006493.html
The jdk9 changeset:
http://hg.openjdk.java.net/jdk9/client/jdk/rev/1490b2b2af97
The jdk8u changeset:
http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/1490b2b2af97
Thanks,
Dmitry
--
Best regards, Sergey.
