On Wed, 11 May 2022 18:25:09 GMT, Phil Race <p...@openjdk.org> wrote:

> This replaces the PR from Toshio  https://git.openjdk.java.net/jdk/pull/8329
> It is similar in the idea from what we converged on towards the end there but
> 1) I'd like to preserve all the support for the old encodings since JEP-400 
> explicitly supports using -Dfile.encoding=windows-31j (for example)
> 2) I wanted to have the look up for the font to use in the windows text 
> control succeed on the first font it finds not the last. In practice I expect 
> this to be a no-op but if someone did have two that match .. don't you think 
> they mean the first one to be the primary ?
> 3) I needed to update the encoding match to ensure that some optional 
> re-ordering for Japanese logical fonts works in the UTF-8.ja locale as that 
> is now the default.
> 
> I verified this fix by changing regional settings as well as passing 
> -Duser.language=ja -Duser.country=JP and logical fonts on lightweights 
> (verified in Font2DTest) are as expected and the "fonttest.java" now behaves 
> as it did in JDK 17 and earlier. Since you need to switch regional settings 
> automating this test was not possible.
> 
> @toshiona you very likely want to take a look at it.

This pull request has now been integrated.

Changeset: 40e99a19
Author:    Phil Race <p...@openjdk.org>
URL:       
https://git.openjdk.java.net/jdk/commit/40e99a19f20dde0916684f93c17c51c7c5de109a
Stats:     27 lines in 1 file changed: 21 ins; 2 del; 4 mod

8285308: Win: Japanese logical fonts are drawn with wrong size

Reviewed-by: kizune, naoto

-------------

PR: https://git.openjdk.java.net/jdk/pull/8662

Reply via email to