Re: [10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer
Looks good to me, too. -- Kevin Laurent Bourgès wrote: Jim, It looks good to me (even I am not an official reviewer) and the Marlin Double-precision will be the default rasterizer. Bye, Laurent Le 14 avr. 2017 22:32, "Jim Graham"a écrit : JBS: https://bugs.openjdk.java.net/browse/JDK-8177985 webrev: http://cr.openjdk.java.net/~flar/JDK-8177985/webrev.00/ The only way to get one of the old Pisces rasterizers now is to manually disable Marlin (or use the new order option as follows). This fix also introduces a similar pattern for rasterizer selection that we use for pipeline selection: -Dprism.rasterizerorder=listof( marlin|marlinfloat|marlindouble| pisces|javapisces|nativepisces| *) (unknown values are ignored for future/backwards-proofing) Suggestions for naming of the new rasterizer identifiers and the "public name" strings are welcome... ...jim
Re: [10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer
Yes... ...jim On 4/14/17 1:45 PM, Michael Paus wrote: Will marlindouble remain the default when you say nothing or when you specify just marlin? Am 14.04.17 um 22:31 schrieb Jim Graham: JBS: https://bugs.openjdk.java.net/browse/JDK-8177985 webrev: http://cr.openjdk.java.net/~flar/JDK-8177985/webrev.00/ The only way to get one of the old Pisces rasterizers now is to manually disable Marlin (or use the new order option as follows). This fix also introduces a similar pattern for rasterizer selection that we use for pipeline selection: -Dprism.rasterizerorder=listof( marlin|marlinfloat|marlindouble| pisces|javapisces|nativepisces| *) (unknown values are ignored for future/backwards-proofing) Suggestions for naming of the new rasterizer identifiers and the "public name" strings are welcome... ...jim
Re: [10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer
Jim, It looks good to me (even I am not an official reviewer) and the Marlin Double-precision will be the default rasterizer. Bye, Laurent Le 14 avr. 2017 22:32, "Jim Graham"a écrit : > JBS: https://bugs.openjdk.java.net/browse/JDK-8177985 > webrev: http://cr.openjdk.java.net/~flar/JDK-8177985/webrev.00/ > > The only way to get one of the old Pisces rasterizers now is to manually > disable Marlin (or use the new order option as follows). > > This fix also introduces a similar pattern for rasterizer selection that > we use for pipeline selection: > > -Dprism.rasterizerorder=listof( > marlin|marlinfloat|marlindouble| > pisces|javapisces|nativepisces| > *) > (unknown values are ignored for future/backwards-proofing) > > Suggestions for naming of the new rasterizer identifiers and the "public > name" strings are welcome... > > ...jim >
Re: [10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer
Will marlindouble remain the default when you say nothing or when you specify just marlin? Am 14.04.17 um 22:31 schrieb Jim Graham: JBS: https://bugs.openjdk.java.net/browse/JDK-8177985 webrev: http://cr.openjdk.java.net/~flar/JDK-8177985/webrev.00/ The only way to get one of the old Pisces rasterizers now is to manually disable Marlin (or use the new order option as follows). This fix also introduces a similar pattern for rasterizer selection that we use for pipeline selection: -Dprism.rasterizerorder=listof( marlin|marlinfloat|marlindouble| pisces|javapisces|nativepisces| *) (unknown values are ignored for future/backwards-proofing) Suggestions for naming of the new rasterizer identifiers and the "public name" strings are welcome... ...jim
Re: [10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer
I was going to tag Kevin and Laurent on this, but forgot to amend the address lines before I hit send... ...jim On 4/14/17 1:31 PM, Jim Graham wrote: JBS: https://bugs.openjdk.java.net/browse/JDK-8177985 webrev: http://cr.openjdk.java.net/~flar/JDK-8177985/webrev.00/ The only way to get one of the old Pisces rasterizers now is to manually disable Marlin (or use the new order option as follows). This fix also introduces a similar pattern for rasterizer selection that we use for pipeline selection: -Dprism.rasterizerorder=listof( marlin|marlinfloat|marlindouble| pisces|javapisces|nativepisces| *) (unknown values are ignored for future/backwards-proofing) Suggestions for naming of the new rasterizer identifiers and the "public name" strings are welcome... ...jim
[10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer
JBS: https://bugs.openjdk.java.net/browse/JDK-8177985 webrev: http://cr.openjdk.java.net/~flar/JDK-8177985/webrev.00/ The only way to get one of the old Pisces rasterizers now is to manually disable Marlin (or use the new order option as follows). This fix also introduces a similar pattern for rasterizer selection that we use for pipeline selection: -Dprism.rasterizerorder=listof( marlin|marlinfloat|marlindouble| pisces|javapisces|nativepisces| *) (unknown values are ignored for future/backwards-proofing) Suggestions for naming of the new rasterizer identifiers and the "public name" strings are welcome... ...jim