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