[15] RFR: 8245241: Incorrect locale provider preference is not logged

2020-05-20 Thread naoto . sato
Hello, Please review the fix to the following issue: https://bugs.openjdk.java.net/browse/JDK-8245241 The proposed changeset is located at: https://cr.openjdk.java.net/~naoto/8245241/webrev.00/ Incorrect user-provided provider preference is supposed to be logged. However it is not so because

[15] RFR: 8239480: Support for CLDR version 37

2020-05-20 Thread naoto . sato
Hello, Please review the fix to the following issue: https://bugs.openjdk.java.net/browse/JDK-8239480 The proposed changeset, and the release note draft are located at: https://cr.openjdk.java.net/~naoto/8239480/webrev.00/ https://bugs.openjdk.java.net/browse/JDK-8243581 This updates the CLDR

Re: [15] RFR: 8245241: Incorrect locale provider preference is not logged

2020-05-20 Thread Joe Wang
Hi Naoto, I don't seem to see the DateFormat class or the getDateInstance methods specify how errors may be handled (or logged). Is that stated somewhere else?  In other cases, I see that you've changed it to throw ServiceConfigurationError, that looks to me may be better than a log as a conf

Re: [15] RFR: 8245241: Incorrect locale provider preference is not logged

2020-05-20 Thread naoto . sato
Hi Joe, thanks for the review. On 5/20/20 4:10 PM, Joe Wang wrote: Hi Naoto, I don't seem to see the DateFormat class or the getDateInstance methods specify how errors may be handled (or logged). Is that stated somewhere else? Description of the system property "java.locale.providers" is in

Re: [15] RFR: 8245241: Incorrect locale provider preference is not logged

2020-05-20 Thread Joe Wang
Hi Naoto, Thanks for the explanation. I agree on the needs for maintaining backward compatibility.  The patch looks good me then. Best, Joe On 5/20/2020 4:49 PM, naoto.s...@oracle.com wrote: Hi Joe, thanks for the review. On 5/20/20 4:10 PM, Joe Wang wrote: Hi Naoto, I don't seem to see t

Re: [15] RFR: 8239480: Support for CLDR version 37

2020-05-20 Thread Joe Wang
Looks good to me. A minor comment: in the license files, Terms of Use lost its original format (e.g. headings and numbering), lines are also super long. But I think we've seen them in the previous update, it's up to you whether you'd want to reformat th

Re: [15] RFR: 8239480: Support for CLDR version 37

2020-05-20 Thread naoto . sato
Hi Joe, thanks again! The license file "unicode-license.txt" is actually included in the version 37 artifact: https://github.com/unicode-org/cldr/blob/release-37/unicode-license.txt The changeset uses the file as is, so I believe it is OK. Naoto On 5/20/20 6:51 PM, Joe Wang wrote: Looks go

Re: [15] RFR: 8239480: Support for CLDR version 37

2020-05-20 Thread Joe Wang
On 5/20/2020 7:06 PM, naoto.s...@oracle.com wrote: Hi Joe, thanks again! No problem. The license file "unicode-license.txt" is actually included in the version 37 artifact: https://github.com/unicode-org/cldr/blob/release-37/unicode-license.txt The changeset uses the file as is, so I b