The fix looks good to me.
Thanks,
Alexandr.
On 8/5/2014 4:34 PM, Sergey Bylokhov wrote:
The fix looks good.
On 05.08.2014 16:17, dmitry markov wrote:
Thank you, Sean.
Alexandr, Sergey,
Could you review the trivial changes (lambda -> anonymous inner
class) in the regression test, please?
Thanks,
Dmitry
On 05/08/2014 13:15, Seán Coffey wrote:
Approved. Have a reviewer review the testcase change before pushing.
Thanks,
Sean.
On 05/08/2014 06:11, dmitry markov wrote:
Hello,
Could you approve the back-port of the fix for 8041990 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-8041990
The webrev:
http://cr.openjdk.java.net/~dmarkov/8041990/jdk7u/webrev.00/
The review thread:
http://mail.openjdk.java.net/pipermail/awt-dev/2014-May/007914.html
The jdk9 chageset:
http://hg.openjdk.java.net/jdk9/client/jdk/rev/5cab75dd1bbd
The jdk8u changeset:
http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ebab5c9c6c76
Thanks,
Dmitry
