RFR: 8307953: [AIX] C locale's font setting was changed by JEP 400 [v2]

Ichiroh Takiguchi itakiguchi at openjdk.org
Tue Jun 6 15:42:53 UTC 2023


On Tue, 6 Jun 2023 15:27:31 GMT, Ichiroh Takiguchi <itakiguchi at openjdk.org> 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

Thanks @prrace and @naotoj .
I appreciate your suggestions.
Now I only touched `src/java.desktop/aix/data/fontconfig/fontconfig.properties`
Could you review new change ?
Thanks in advance.

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

PR Comment: https://git.openjdk.org/jdk/pull/13937#issuecomment-1579005703



More information about the client-libs-dev mailing list