On Tue, 6 Jun 2023 15:27:31 GMT, Ichiroh Takiguchi <[email protected]> wrote:
>> On AIX, lib/fontconfig.bfc file is used to find font setting. >> I ran SwingSet2 List demo program with AIX C locale. >> JDK21's line spacing is larger then JDK17. >> Screen shots are in >> [JDK-8307953](https://bugs.openjdk.org/browse/JDK-8307953). >> When I used `-Dfile.encoding=COMPAT` option on AIX C locale, SwingSet2 >> worked fine as expected. >> >> This situation happens `Charset.defaultCharset()` always returns UTF-8 after >> JEP 400 by default. >> `native.encoding` system property has platform's encoding/charset name. >> If platform's charset name is not supported by Java, charset name should be >> `UTF-8`. >> >> Additionally, following parts are changed: >> `src/java.desktop/aix/data/fontconfig/fontconfig.propertie`s has invalid >> charset name. >> For Java for AIX: >> - Encoding name for AIX's Zh_TW locale is `big5`, charset name should be >> `x-IBM950` instead of `big5`. >> - Encoding name for AIX's zh_CN locale is `IBM-eucCN`, charset name should >> be `x-IBM1383` instead of `x-EUCCN`. >> >> In my understanding, this fix affects just for AIX platform. >> >> Note: >> macos-x64 tier1 test was failed, but I think it's not related this PR. > > Ichiroh Takiguchi has updated the pull request incrementally with one > additional commit since the last revision: > > 8307953: [AIX] C locale's font setting was changed by JEP 400 The part reverting the encoding looks good to me. I defer the other part review for client team. ------------- Marked as reviewed by naoto (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/13937#pullrequestreview-1465968534
