Re: [12]: RFR: 8209167: Use CLDR's time zone mappings for Windows

2018-09-13 Thread Magnus Ihse Bursie
On 2018-09-12 19:33, naoto.s...@oracle.com wrote: Hi Magnus, Erik, Thank you for your comments. I updated the webrev according to your suggestions: http://cr.openjdk.java.net/~naoto/8209167/webrev.02/ Looks good to me. /Magnus As to the duplicated "common" in the dependency, yes

Re: [12]: RFR: 8209167: Use CLDR's time zone mappings for Windows

2018-09-12 Thread Erik Joelsson
Looks good. /Erik On 2018-09-12 10:33, naoto.s...@oracle.com wrote: Hi Magnus, Erik, Thank you for your comments. I updated the webrev according to your suggestions: http://cr.openjdk.java.net/~naoto/8209167/webrev.02/ As to the duplicated "common" in the dependency, yes that's a

Re: [12]: RFR: 8209167: Use CLDR's time zone mappings for Windows

2018-09-12 Thread naoto . sato
Hi Magnus, Erik, Thank you for your comments. I updated the webrev according to your suggestions: http://cr.openjdk.java.net/~naoto/8209167/webrev.02/ As to the duplicated "common" in the dependency, yes that's a separate issue. Since it's obvious, I included the fix with this changeset (it