<AWT Dev> Review request for JDK-8038631: Create wrapper for awt.Robot with additional functionality
Petr Pchelko
petr.pchelko at oracle.com
Fri Apr 4 15:11:02 UTC 2014
And it would be good to use the GetIntegerAction with the default and get rid of the if clause.
With best regards. Petr.
On 04.04.2014, at 19:09, Dmitriy Ermashov <dmitriy.ermashov at oracle.com> wrote:
> Oops, small typo!
> Will fix without webrev, ok?
>
> With regards,
> Dmitriy Ermashov
>
> 04 апр. 2014 г., в 19:06, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> написал(а):
>
>> On 4/4/14 7:02 PM, Dmitriy Ermashov wrote:
>>> Hi,
>>> Please, review the changeset for:
>>> https://bugs.openjdk.java.net/browse/JDK-8038631
>>>
>>> Webrev is here:
>>> http://cr.openjdk.java.net/~yan/8038631/webrev.05/
>>>
>>> Last changes consist of:
>>> 1. Modified property read process, added sun.security.action.GetIntegerAction usage.
>>> 2. Javadoc improvements, added @link, rephrased some sentences.
>>> 3. Refactored overloaded glide() methods.
>> Nice refactoring "GetIntegerAction1".
>>
>>
>> --
>> Best regards, Sergey.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20140404/3f71605b/attachment.html>
More information about the awt-dev
mailing list