<AWT Dev> [9] Review Request: 8032435 Cleanup of sun.awt.windows package
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Mon Jan 27 03:42:25 PST 2014
I added @Override, because it make code refactoring safer.
I have separate CR for lamdafication
https://bugs.openjdk.java.net/browse/JDK-8031781
On 27.01.2014 14:42, Artem Ananiev wrote:
>
> Since you're adding @Override in many places, didn't you think about
> replacing some of the inner Runnable classes with lambdas?
>
> Thanks,
>
> Artem
>
> On 1/24/2014 6:49 PM, Sergey Bylokhov wrote:
>> Hello.
>> Please review the fix for jdk 9.
>> This is cleanup of the sun.awt.windows package:
>> - Encapsulation strengthening.
>> - final and @Override were added whenever possible.
>> - WPrintDialog now uses AWTAccessor instead of its own setPeer()
>> native method.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8032435
>> Webrev can be found at:
>> http://cr.openjdk.java.net/~serb/8032435/webrev.00
>>
--
Best regards, Sergey.
More information about the awt-dev
mailing list