<AWT Dev> [8] Review Request: 8022657 Add FunctionalInterface annotation to awt interfaces

Anthony Petrov anthony.petrov at oracle.com
Thu Oct 17 13:21:52 PDT 2013


+1

--
best regards,
Anthony

On 10/17/2013 06:55 PM, Artem Ananiev wrote:
>
> 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
>>>>
>>
>>


More information about the awt-dev mailing list