<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