[jdk8u-dev] RFR: 8235834: IBM-943 charset encoder needs updating [v4]
Severin Gehwolf
sgehwolf at openjdk.org
Wed May 15 12:55:08 UTC 2024
On Mon, 13 May 2024 12:58:21 GMT, Pratiksha Sawant <duke at openjdk.org> wrote:
>> Backporting JDK-8235834 - IBM-943 charset encoder needs updating.
>> There are few characters which display garbled values when using IBM943C, so this mapping file needs to be backported to fix the issue.
>>
>> Reported issue : https://bugs.openjdk.org/browse/JDK-8235834
>
> Pratiksha Sawant has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - Merge branch 'openjdk:master' into IBM943-support-backport-1b9251
> - correcting comment for clean build
> - Correcting comments for clean backport
> - Backport 1b925183894778acce8aff4c101f67af98f0097f
>
> Signed-off-by: Pratiksha.Sawant <Pratiksha.Sawant at ibm.com>
The test fails:
/disk/openjdk/upstream-sources/git/jdk8u-dev/jdk/test/sun/nio/cs/TestIBMBugs.java:223: error: cannot find symbol
if (!Arrays.equals(byteIBM943c2b, strIBM943c2b1.getBytes(cs))) {
^
symbol: variable Arrays
location: class TestIBMBugs
1 error
result: Failed. Compilation failed: Compilation failed
Adding the import should fix it.
Also, please enable GHA for your fork and show some evidence on what testing you've done for this. Thanks!
-------------
Changes requested by sgehwolf (Reviewer).
PR Review: https://git.openjdk.org/jdk8u-dev/pull/476#pullrequestreview-2057922151
More information about the jdk8u-dev
mailing list