> 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.

Phil Race has updated the pull request incrementally with one additional commit 
since the last revision:

  8285308

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8662/files
  - new: https://git.openjdk.java.net/jdk/pull/8662/files/49d5465e..79d246bc

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8662&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8662&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8662.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8662/head:pull/8662

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

Reply via email to