Other than the copyright year things, all looks good to me. I don't think further review is required for copyright update.

Masayoshi

On 5/28/2016 3:25 AM, Naoto Sato wrote:
Hi Rachna,

Here are my comments to the webrev (I am assuming the tool that extracts JavaTime*.java are working correctly, i.e., have not reviewed the data itself):

- (All JavaTime*.java files): Unicode copyright notice is outdated. It should include the one that has "1991-2016" as the copyright year. So as the file "unicode-license.txt"

- JavaTimeSupplementary_iw.java has the copyright year of "2016,". It should be "2013, 2016,".

Others look OK.

Naoto

On 5/26/16 4:30 AM, Rachna Goel wrote:
Hi all,

Please review fix for JDK-8145136.

Bug: https://bugs.openjdk.java.net/browse/JDK-8145136

webrev:
http://cr.openjdk.java.net/~nishjain/rachna/CLDR29/8145136/webrev.00/

Fix: Currently JDK supports CLDR V27 which is upgraded to latest version
29.

For more info:  http://cldr.unicode.org/


Reply via email to