The fix looks good to me.

 There is a small comment about code formatting.
I believe that according to the new Java Style Guidelines where should not be a space between open brackets "( ()"
http://cr.openjdk.java.net/~alundblad/styleguide/#toc-lambda-expressions

 Thanks,
 Alexandr.

On 3/1/2016 2:18 AM, Renaud Paquay wrote:
looks good to me -- assuming it is ok to use lambda syntax in this file, the rest of the file still uses the anonymous inner class syntax.

On Mon, Feb 29, 2016 at 12:35 PM, Pete Brunet <peter.bru...@oracle.com <mailto:peter.bru...@oracle.com>> wrote:

    Please review this change which runs code on the EDT like it
    should have.

    http://cr.openjdk.java.net/~ptbrunet/JDK-8145228/webrev.00/
    <http://cr.openjdk.java.net/%7Eptbrunet/JDK-8145228/webrev.00/>



Reply via email to