[OpenJDK 2D-Dev] RFR 8199789: Emit a warning message when t2k is selected via system property

[Phil Race]

I typed

+ (!(FontUtilities.useT2K && !FontUtilities.useLegacy) ? when I actually 
meant + ((!FontUtilities.useT2K && !FontUtilities.useLegacy) ? ie the 
first "!" was supposed to be inside ..


I decided to add a debugging option to make it easier to tell what you'd got
and then further decided to add code that falls back to freetype if you
asked for T2K but it isn't there.


http://cr.openjdk.java.net/~prr/8199789.1

I've tested it with openjdk + oracle JDK builds.

I think it'll do for now until T2K is removed.


-phil.
On 03/20/2018 02:30 AM, Prahalad Kumar Narayanan wrote:
> Hello Phil
>
> Good day to you.
>
> I imported your patch and checked the resulting build.
> Warnings showed up as expected with use of "t2k" or "legacy" in the VM option -Dsun.java2d.font.scaler.
>
> However, I'm unable to interpret this line and its intended outcome.
>> It also adds a new value for sun.java2d.font.scaler of "legacy" which means "t2k" but as it was used in by default.
> In my observation with logs, setting VM option to
>      "t2k" instantiates T2KFontScaler and
>      "legacy" instantiates FreetypeFontScaler.
> If this is the intended behavior, the code changes work as expected.
>
> Thank you
> Have a good day
>
> Prahalad N.
>
>
> -----Original Message-----
> From: Phil Race
> Sent: Tuesday, March 20, 2018 1:09 AM
> To: 2d-dev
> Subject: [OpenJDK 2D-Dev] RFR 8199789: Emit a warning message when t2k is selected via system property
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8199789
> Webrev : http://cr.openjdk.java.net/~prr/8199789/
>
> https://bugs.openjdk.java.net/browse/JDK-8193017 made freetype the default font rasteriser for all JDK builds.
>
> We plan to remove t2k completely including references to it from open sources, before JDK 11 GA's
>
> But for now it is still there for debugging but to make it clear this small fix makes a warning get printed if you try to use it.
>
> It also adds a new value for sun.java2d.font.scaler of "legacy" which means "t2k" but as it was used in by default.
>
> The subtle issue is that "t2k" disables using GDI for LCD text.
> "legacy" gets you exactly what JDK did by default in 6u10 -> JDK 10 inclusive.
>
> So it may be more useful for a debugging comparison flag.
>
> -phil.
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20180320/92083c50/attachment.html>