Ok. 

The fix looks good to me.

With best regards. Petr.

On 17.03.2014, at 13:18, Sergey Bylokhov <sergey.bylok...@oracle.com> wrote:

> On 3/17/14 1:15 PM, Petr Pchelko wrote:
>> Hello, Sergey.
>> 
>> Why didn't you replace inner classes with lambdas?
>> AppletPane:641 - you could get rid of the explicit array creation, the 
>> method is vararg.
> That's because this is p5 bug and I don't want to make a change more than a 
> simple cleanup/formatting in it.
>> 
>> With best regards. Petr.
>> 
>> On 17.03.2014, at 13:04, Sergey Bylokhov <sergey.bylok...@oracle.com> wrote:
>> 
>>> Hello.
>>> Please review the fix for jdk 9.
>>> - typo fixed
>>> - small cleanup
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8028050
>>> Webrev can be found at: http://cr.openjdk.java.net/~serb/8028050/webrev.00
>>> 
>>> -- 
>>> Best regards, Sergey.
>>> 
> 
> 
> -- 
> Best regards, Sergey.
> 

Reply via email to