Code Review for open

Prepared by:naoto on Thu Nov 2 13:32:56 PDT 2017
Workspace:/projects/jdk/10/ldml/open
Compare against: http://hg.openjdk.java.net/jdk10/master
Compare against version:47479
Summary of changes: 2681 lines changed: 2384 ins; 146 del; 151 mod; 30737 unchg
Changeset: open.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw make/jdk/src/classes/build/tools/cldrconverter/CLDRConverter.java 

rev 47480 : [mq]: 8176841
149 lines changed: 59 ins; 76 del; 14 mod; 928 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw make/jdk/src/classes/build/tools/cldrconverter/LDMLParseHandler.java 

rev 47480 : [mq]: 8176841
76 lines changed: 43 ins; 18 del; 15 mod; 907 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw make/jdk/src/classes/build/tools/cldrconverter/NumberingSystemsParseHandler.java 

rev 47480 : [mq]: 8176841
27 lines changed: 23 ins; 0 del; 4 mod; 75 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw make/jdk/src/classes/build/tools/cldrconverter/ResourceBundleGenerator.java 

rev 47480 : [mq]: 8176841
39 lines changed: 22 ins; 1 del; 16 mod; 335 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/text/DateFormat.java 

rev 47480 : [mq]: 8176841
7 lines changed: 7 ins; 0 del; 0 mod; 1062 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/text/DateFormatSymbols.java 

rev 47480 : [mq]: 8176841
11 lines changed: 8 ins; 0 del; 3 mod; 880 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/text/DecimalFormatSymbols.java 

rev 47480 : [mq]: 8176841
13 lines changed: 10 ins; 0 del; 3 mod; 904 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/text/NumberFormat.java 

rev 47480 : [mq]: 8176841
8 lines changed: 7 ins; 0 del; 1 mod; 1252 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/text/SimpleDateFormat.java 

rev 47480 : [mq]: 8176841
1 line changed: 0 ins; 0 del; 1 mod; 2501 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/time/format/DateTimeFormatter.java 

rev 47480 : [mq]: 8176841
36 lines changed: 31 ins; 0 del; 5 mod; 2203 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/time/format/DateTimeFormatterBuilder.java 

rev 47480 : [mq]: 8176841
27 lines changed: 24 ins; 0 del; 3 mod; 4934 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/time/format/DateTimeTextProvider.java 

rev 47480 : [mq]: 8176841
3 lines changed: 2 ins; 0 del; 1 mod; 590 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/time/temporal/WeekFields.java 

rev 47480 : [mq]: 8176841
6 lines changed: 5 ins; 1 del; 0 mod; 1165 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/util/Calendar.java 

rev 47480 : [mq]: 8176841
23 lines changed: 17 ins; 0 del; 6 mod; 3627 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/util/Currency.java 

rev 47480 : [mq]: 8176841
33 lines changed: 27 ins; 1 del; 5 mod; 972 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/util/Locale.java 

rev 47480 : [mq]: 8176841
189 lines changed: 91 ins; 47 del; 51 mod; 3321 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/util/spi/LocaleNameProvider.java 

rev 47480 : [mq]: 8176841
52 lines changed: 51 ins; 0 del; 1 mod; 143 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/launcher/LauncherHelper.java 

rev 47480 : [mq]: 8176841
1 line changed: 0 ins; 0 del; 1 mod; 1291 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/cldr/CLDRLocaleProviderAdapter.java 

rev 47480 : [mq]: 8176841
14 lines changed: 12 ins; 1 del; 1 mod; 221 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/CalendarDataUtility.java 

rev 47480 : [mq]: 8176841
55 lines changed: 51 ins; 0 del; 4 mod; 204 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/DateFormatProviderImpl.java 

rev 47480 : [mq]: 8176841
16 lines changed: 13 ins; 0 del; 3 mod; 166 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/LocaleDataMetaInfo.java 

rev 47480 : [mq]: 8176841
12 lines changed: 11 ins; 0 del; 1 mod; 48 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/LocaleNameProviderImpl.java 

rev 47480 : [mq]: 8176841
21 lines changed: 20 ins; 0 del; 1 mod; 182 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/NumberFormatProviderImpl.java 

rev 47480 : [mq]: 8176841
8 lines changed: 5 ins; 0 del; 3 mod; 224 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/SPILocaleProviderAdapter.java 

rev 47480 : [mq]: 8176841
15 lines changed: 14 ins; 0 del; 1 mod; 615 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/locale/provider/TimeZoneNameUtility.java 

rev 47480 : [mq]: 8176841
16 lines changed: 15 ins; 0 del; 1 mod; 265 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/sun/util/resources/LocaleNames.properties 

rev 47480 : [mq]: 8176841
3 lines changed: 0 ins; 1 del; 2 mod; 1168 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/util/Calendar/Bug4302966.java 

rev 47480 : [mq]: 8176841
3 lines changed: 0 ins; 0 del; 3 mod; 40 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/tools/jlink/plugins/IncludeLocalesPluginTest.java 

rev 47480 : [mq]: 8176841
37 lines changed: 36 ins; 0 del; 1 mod; 514 unchg

------ ------ ------ ------ --- New ----- Raw make/jdk/src/classes/build/tools/cldrconverter/TimeZoneParseHandler.java 

rev 47480 : [mq]: 8176841
64 lines changed: 64 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw src/jdk.localedata/share/classes/sun/util/cldr/resources/common/bcp47/timezone.xml 

rev 47480 : [mq]: 8176841
470 lines changed: 470 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw src/jdk.localedata/share/classes/sun/util/cldr/resources/common/dtd/ldmlBCP47.dtd 

rev 47480 : [mq]: 8176841
74 lines changed: 74 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/CalendarTests.java 

rev 47480 : [mq]: 8176841
156 lines changed: 156 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/CurrencyTests.java 

rev 47480 : [mq]: 8176841
106 lines changed: 106 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/DefaultLocaleTest.java 

rev 47480 : [mq]: 8176841
46 lines changed: 46 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/DisplayNameTests.java 

rev 47480 : [mq]: 8176841
104 lines changed: 104 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/FormatTests.java 

rev 47480 : [mq]: 8176841
167 lines changed: 167 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/JavaTimeTests.java 

rev 47480 : [mq]: 8176841
259 lines changed: 259 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/SymbolsTests.java 

rev 47480 : [mq]: 8176841
91 lines changed: 91 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/SystemPropertyTests.java 

rev 47480 : [mq]: 8176841
99 lines changed: 99 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/spi/LocaleNameProviderTests.java 

rev 47480 : [mq]: 8176841
52 lines changed: 52 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/spi/provider/foo/LocaleNameProviderImpl.java 

rev 47480 : [mq]: 8176841
65 lines changed: 65 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/jdk/java/util/Locale/bcp47u/spi/provider/module-info.java 

rev 47480 : [mq]: 8176841
27 lines changed: 27 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /projects/jdk/code-tools/webrev/webrev.ksh (vers 25.16-hg+openjdk.java.net).