RFR: 8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform [v2]

KIRIYAMA Takuya duke at openjdk.org
Thu Sep 29 08:08:16 UTC 2022


On Wed, 28 Sep 2022 18:00:09 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> KIRIYAMA Takuya has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform
>
> test/jdk/tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java line 55:
> 
>> 53:         String encoding = System.getProperty("native.encoding");
>> 54:         switch (encoding) {
>> 55:         default:
> 
> What I meant by the previous comment was to replace this `default` clause to:
> 
>     case "Cp1252", "UTF-8" -> testString = new String(Character.toChars(0x00E9));
> 
> And for other unknown encodings:
> 
>     default -> {
>         System.out.println("Test skipped"); // or better message
>         return;
>     }
> 
> And your fix:
> 
>     case "MS932", "SJIS" -> testString = new String(Character.toChars(0x3042));
> 
> This way only the encodings that are guaranteed to succeed are tested.

I am sorry to misunderstand your comment. 

I don’t know whether "é"(0x00e9) exists only in "Cp1252", "UTF-8".
Will that be no problem that this test is skipped in other encordings? If skipped, the pattern for its encording probably won't be added forever. I think the other pattern should be added when this test fails. However I will follow the policy of OpenJDK.

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

PR: https://git.openjdk.org/jdk/pull/10226


More information about the core-libs-dev mailing list