Looks fine.
Thanks,
Artem
On 10/17/2013 6:34 PM, Sergey Bylokhov wrote:
Hi, Artem.
Here is updated version.
http://cr.openjdk.java.net/~serb/8022657/webrev.01
On 17.10.2013 18:10, Artem Ananiev wrote:
Hi, Sergey,
we have a number of listener interfaces in java.awt.event, some of
them have a single method, others have multiple. For consistency, I
would refrain from marking some of them as @FunctionalInterfaces
(though it's technically possible).
KeyEventDispatcher and KeyEventPostProcessor are fine.
Thanks,
Artem
On 10/16/2013 8:03 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 8.
Some of the classes in awt were marked as FunctionalInterface.
List of all suggested classes
http://mail.openjdk.java.net/pipermail/awt-dev/2013-February/004213.html
Bug: https://bugs.openjdk.java.net/browse/JDK-8022657
Webrev can be found at:
http://cr.openjdk.java.net/~serb/8022657/webrev.00