RFR JDK-8186801: Add regression test to test mapping based charsets (generated at build time)

Xueming Shen xueming.shen at oracle.com
Mon Aug 28 06:00:00 UTC 2017


Hi,

Please help codereview the changes for JDK-8186801 and JDK-8186803

issue: https://bugs.openjdk.java.net/browse/JDK-8186801
           https://bugs.openjdk.java.net/browse/JDK-8186803
webrev: http://cr.openjdk.java.net/~sherman/8186801/webrev

Notes:

JDK-8186801:
During the discussion of adding ISO-8859-16 charset into jdk10 it 
appears it is
really inconvenient not having the appropriate regression/unit tests to 
verify the
correctness of charset mapping in open repo.  The proposed change here is to
migrate one of the corresponding test (sun.nio.cs.TestCoder.java) in our 
closed
repo to open repo sun.nio.cs.TestCharsetMapping.java.

The newly added test verifies those "gensrc" charsets against the 
mappings in
make/data/charsetmapping for the charset name, aliases and the correctness
of the mappings, which triggers couple minor changes

(1) x-SJIS_0213/x-MS932_0213, wrong charset name for aliases lookup.
(2) euc-jp-open, inconsistent charset name
(3) change mapping table file name from euc_tw to EUC_TW
(4) migrated couple mapping tables from closed repo to open repo to test
      those "template" based gensrc charsets (we don't have mapping tables
      in repo now)
(5) and inconsistent handling of  linefeed for charset cp1140-1149, which
      is recorded in JDK-8186803.

JDK-8186803:

All JDK's EBCDIC charsets, except Cp1140-1149, map the line-feed
\u000A character to EBCDIC 0x15. The related mappings are as

0x25 (LF) => U+000A
0x15(NL) <=> U+000A
0x15(NL) <= U+0085

Cp1140-1149 however maps the U+000A.

As suggested in JDK-7016785, there are/were two standards to handle the 
newline
in z/OS ( LF/0x25 CDRA or NL/0x15). From JDK1.4, IBM idk has moved on to map
\u000a to 0x15 as the default, which matches the behavior of most our ebcdic
charsets.

This rfe is to update the Cp114[0-9] to also map \u000A to ebcdic 0x15 to be
consistent with other ebcdic charsets (to add non-roundtrip mapping 
tables for
IBM0114[0-9])

Thanks
Sherman



More information about the nio-dev mailing list