RFR: 8214533 IBM-29626C is required for AIX default charset

Ichiroh Takiguchi takiguc at linux.vnet.ibm.com
Wed Jan 23 12:21:15 UTC 2019


Hello.

Could you review the fix and give your suggestion ?
Thanks,
Ichiroh Takiguchi

On 2019-01-16 21:32, Ichiroh Takiguchi wrote:
> Hello Alan and Roger.
> 
> I appreciate your suggestions.
> 
> Could you review the fix again ?
> 
> Bug:    https://bugs.openjdk.java.net/browse/JDK-8214533
> Change: https://cr.openjdk.java.net/~itakiguchi/8214533/webrev.02/
> 
> I used "hg cp" command.
> If I should use git format, please let me know.
> 
> I changed Copyright year on IBM29626C.java.template and charsets file.
> 
> Thanks,
> Ichiroh Takiguchi
> 
> On 2019-01-16 01:01, Roger Riggs wrote:
>> Hi,
>> 
>> I'd suggest using 2002, 2019, for the copyright, since much of the
>> code in the new
>> file comes from an older source.
>> 
>> $.02, Roger
>> 
>> 
>> On 01/15/2019 10:43 AM, Alan Bateman wrote:
>>> On 15/01/2019 00:51, Ichiroh Takiguchi wrote:
>>>> Hello Alan.
>>>> 
>>>> Could you review the fix again ?
>>>> 
>>>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8214533
>>>> Change: https://cr.openjdk.java.net/~itakiguchi/8214533/webrev.01/
>>>> 
>>>> I added IBM29626C charset as standard way.
>>>> Please give any suggestion and question.
>>> This looks okay, assuming it's `hg copy` of EUR_JP.template to create 
>>> IBM29626C.java.template rather than a `hg mv` (the webrev makes it 
>>> look like a move/rename).
>>> 
>>> -Alan



More information about the core-libs-dev mailing list