<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    I typed<br>
    <pre><span class="new">+                (!(FontUtilities.useT2K && !FontUtilities.useLegacy) ?

when I actually meant

</span><span class="new">+                ((!FontUtilities.useT2K && !FontUtilities.useLegacy) ?

ie the first "!" was supposed to be inside .. 
</span></pre>
    <pre><span class="new"></span>
<span class="new">I decided to add a debugging option to make it easier to tell what you'd got</span>
<span class="new">and then further decided to add code that falls back to freetype if you</span>
<span class="new">asked for T2K but it isn't there.</span>
<span class="new"></span>
<span class="new"></span>
<span class="new"><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~prr/8199789.1">http://cr.openjdk.java.net/~prr/8199789.1</a></span>
<span class="new"></span>
<span class="new">I've tested it with openjdk + oracle JDK builds.</span>
<span class="new"></span>
<span class="new">I think it'll do for now until T2K is removed.</span>
<span class="new"></span></pre>
    <span class="new"><br>
      -phil.<br>
    </span>
    <div class="moz-cite-prefix">On 03/20/2018 02:30 AM, Prahalad Kumar
      Narayanan wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:a26d2e94-2cd4-4ebe-b7de-da44e1401143@default">
      <pre wrap="">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. 
</pre>
      <blockquote type="cite">
        <pre wrap="">It also adds a new value for sun.java2d.font.scaler of "legacy" which means "t2k" but as it was used in by default.
</pre>
      </blockquote>
      <pre wrap="">
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: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8199789">https://bugs.openjdk.java.net/browse/JDK-8199789</a>
Webrev : <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~prr/8199789/">http://cr.openjdk.java.net/~prr/8199789/</a>

<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8193017">https://bugs.openjdk.java.net/browse/JDK-8193017</a> 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.


</pre>
    </blockquote>
    <br>
  </body>
</html>