Code Review for open

Prepared by:naoto on Fri Nov 10 15:31:06 PST 2017
Workspace:/projects/jdk/10/ldml/open
Compare against: ssh://hg.openjdk.java.net/jdk/jdk
Compare against version:qparent
Summary of changes: 3528 lines changed: 2661 ins; 526 del; 341 mod; 40632 unchg
Patch of changes: open.patch
Legend: Modified file
Deleted file
New file

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

rev 47734 : imported patch 8190918
234 lines changed: 63 ins; 138 del; 33 mod; 847 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
76 lines changed: 43 ins; 18 del; 15 mod; 907 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
27 lines changed: 23 ins; 0 del; 4 mod; 75 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
39 lines changed: 22 ins; 1 del; 16 mod; 335 unchg

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

rev 47734 : imported patch 8190918
71 lines changed: 26 ins; 35 del; 10 mod; 132 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
7 lines changed: 7 ins; 0 del; 0 mod; 1062 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
11 lines changed: 8 ins; 0 del; 3 mod; 880 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
13 lines changed: 10 ins; 0 del; 3 mod; 904 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
8 lines changed: 7 ins; 0 del; 1 mod; 1252 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 2501 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
36 lines changed: 31 ins; 0 del; 5 mod; 2203 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
27 lines changed: 24 ins; 0 del; 3 mod; 4934 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
3 lines changed: 2 ins; 0 del; 1 mod; 590 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
6 lines changed: 5 ins; 1 del; 0 mod; 1165 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
23 lines changed: 17 ins; 0 del; 6 mod; 3627 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
33 lines changed: 27 ins; 1 del; 5 mod; 972 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
190 lines changed: 92 ins; 47 del; 51 mod; 3321 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
52 lines changed: 51 ins; 0 del; 1 mod; 143 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 1291 unchg

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

rev 47734 : imported patch 8190918
34 lines changed: 32 ins; 1 del; 1 mod; 221 unchg

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

rev 47734 : imported patch 8190918
5 lines changed: 2 ins; 0 del; 3 mod; 65 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
55 lines changed: 51 ins; 0 del; 4 mod; 204 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
16 lines changed: 13 ins; 0 del; 3 mod; 166 unchg

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

rev 47734 : imported patch 8190918
2 lines changed: 0 ins; 0 del; 2 mod; 549 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
12 lines changed: 11 ins; 0 del; 1 mod; 48 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
21 lines changed: 20 ins; 0 del; 1 mod; 182 unchg

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

rev 47734 : imported patch 8190918
8 lines changed: 0 ins; 2 del; 6 mod; 511 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
8 lines changed: 5 ins; 0 del; 3 mod; 224 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
15 lines changed: 14 ins; 0 del; 1 mod; 615 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
16 lines changed: 15 ins; 0 del; 1 mod; 265 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
3 lines changed: 0 ins; 1 del; 2 mod; 1168 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
3 lines changed: 0 ins; 0 del; 3 mod; 40 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/sun/text/resources/LocaleData.cldr

rev 47734 : imported patch 8190918
147 lines changed: 0 ins; 0 del; 147 mod; 8234 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/sun/text/resources/LocaleDataTest.java

rev 47734 : imported patch 8190918
1 line changed: 0 ins; 0 del; 1 mod; 490 unchg

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

rev 47734 : imported patch 8190918
43 lines changed: 37 ins; 3 del; 3 mod; 509 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
64 lines changed: 64 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/java.base/share/classes/sun/util/cldr/CLDRCalendarDataProviderImpl.java

rev 47734 : imported patch 8190918
106 lines changed: 106 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
470 lines changed: 470 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
74 lines changed: 74 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/java/util/Calendar/CalendarDataTest.java

rev 47734 : imported patch 8190918
108 lines changed: 108 ins; 0 del; 0 mod; 0 unchg

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

rev 47734 : imported patch 8190918
158 lines changed: 158 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
106 lines changed: 106 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
46 lines changed: 46 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
109 lines changed: 109 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
167 lines changed: 167 ins; 0 del; 0 mod; 0 unchg

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

rev 47734 : imported patch 8190918
261 lines changed: 261 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
91 lines changed: 91 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
99 lines changed: 99 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
52 lines changed: 52 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
65 lines changed: 65 ins; 0 del; 0 mod; 0 unchg

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

rev 47733 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
27 lines changed: 27 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- Patch --- test/java/util/Calendar/Bug8185841.java

rev 47734 : imported patch 8190918
278 lines changed: 0 ins; 278 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).