[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Re: Review request backport 8210386
From:       Laurent_Bourgès <bourges.laurent () gmail ! com>
Date:       2018-10-30 6:19:37
Message-ID: CAKjRUT7F-_pNritoN8N3UDWA-moLcRBdWRAuGf=-1AKhsn788A () mail ! gmail ! com
[Download RAW message or body]

Thank you,

Laurent

Le lun. 29 oct. 2018 =C3=A0 19:35, Kevin Rushforth <kevin.rushforth@oracle.=
com>
a =C3=A9crit :

> Looks good. Approved for pushing to 11-dev.
>
> -- Kevin
>
> On 10/29/2018 11:27 AM, Johan Vos wrote:
> > new webrev including test:
> > http://cr.openjdk.java.net/~jvos/8210386/webrev.01/
> > <http://cr.openjdk.java.net/%7Ejvos/8210386/webrev.00/>
> >
> > On Mon, Oct 29, 2018 at 5:55 PM Kevin Rushforth
> > <kevin.rushforth@oracle.com <mailto:kevin.rushforth@oracle.com>> wrote:
> >
> >     The newly added test, which was pushed to jfx-dev as part of the
> >     fix for
> >     12, is missing from the webrev:
> >
> >     tests/system/src/test/java/test/com/sun/marlin/ScaleClipTest.java
> >
> >     The rest looks OK.
> >
> >     -- Kevin
> >
> >
> >     On 10/29/2018 9:39 AM, Johan Vos wrote:
> >     > Hi Kevin,
> >     >
> >     > Please review the backport of 8210386 (Marlin fixes) into 11-dev
> >     (target
> >     > 11.0.1)
> >     > http://cr.openjdk.java.net/~jvos/8210386/webrev.00/
> >     <http://cr.openjdk.java.net/%7Ejvos/8210386/webrev.00/>
> >     >
> >     > - Johan
> >
>
>
[prev in list] [next in list] [prev in thread] [next in thread]