Re: [10] Review request: 8177985 Use Marlin renderer as the default FX rasterizer

[Kevin Rushforth]

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

[Jim Graham]

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

[Laurent Bourgès]

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

[Michael Paus]

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 Graham]

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

[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