Re: RFR: 6614237: missing codepage Cp290 at java runtime

2013-08-08 Thread Masayoshi Okutsu
Looks good. Masayoshi On 8/9/2013 4:19 AM, Xueming Shen wrote: Thanks Masayoshi, good catch, yes, those two aliases should be included. Webrev has been updated according. webrev: http://cr.openjdk.java.net/~sherman/6614237/webrev/ -Sherman On 08/04/2013 10:17 PM, Masayoshi Okutsu wrote: Hi

Re: RFR: 6614237: missing codepage Cp290 at java runtime

2013-08-08 Thread Xueming Shen
Thanks Masayoshi, good catch, yes, those two aliases should be included. Webrev has been updated according. webrev: http://cr.openjdk.java.net/~sherman/6614237/webrev/ -Sherman On 08/04/2013 10:17 PM, Masayoshi Okutsu wrote: Hi Sherman, IANA seems to define the following aliases for IBM290.

Re: RFR: 6614237: missing codepage Cp290 at java runtime

2013-08-04 Thread Masayoshi Okutsu
Hi Sherman, IANA seems to define the following aliases for IBM290. cp290 EBCDIC-JP-kana csIBM290 The alias table should include EBCDIC-JP-kana and csIBM290 as well? Otherwise, the changes look good to me. Thanks, Masayoshi On 8/3/2013 5:53 AM, Xueming Shen wrote: Masayoshi, Would you pleas

RFR: 6614237: missing codepage Cp290 at java runtime

2013-08-02 Thread Xueming Shen
Masayoshi, Would you please help review this change? The changeset also adds the requested Cp300. webrev: http://cr.openjdk.java.net/~sherman/6614237/webrev/ original mapptables: http://cr.openjdk.java.net/~sherman/6614237/290/ http://cr.openjdk.java.net/~sherman/6614237/300/ regression tests