Re: Review request backport 8210386

2018-10-30 Thread Laurent Bourgès
Thank you, Laurent Le lun. 29 oct. 2018 à 19:35, Kevin Rushforth a écrit : > 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/ > >

Re: Review request backport 8210386

2018-10-29 Thread Kevin Rushforth
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/ On Mon, Oct 29, 2018 at 5:55 PM Kevin Rushforth

Re: Review request backport 8210386

2018-10-29 Thread Johan Vos
new webrev including test: http://cr.openjdk.java.net/~jvos/8210386/webrev.01/ On Mon, Oct 29, 2018 at 5:55 PM Kevin Rushforth wrote: > The newly added test, which was pushed to jfx-dev as part of the fix for > 12, is missing from the

Re: Review request backport 8210386

2018-10-29 Thread Kevin Rushforth
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

Review request backport 8210386

2018-10-29 Thread Johan Vos
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/ - Johan