Code Review for open

Prepared by:naoto on Mon Nov 27 10:27:13 PST 2017
Workspace:/projects/jdk/10/ldml/open
Compare against: ssh://hg.openjdk.java.net/jdk/jdk
Compare against version:qparent
Summary of changes: 4184 lines changed: 3291 ins; 550 del; 343 mod; 42492 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
240 lines changed: 69 ins; 136 del; 35 mod; 847 unchg

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

rev 47935 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
75 lines changed: 42 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 47935 : 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 47935 : 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47935 : 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode 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 47935 : 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 47935 : 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
55 lines changed: 53 ins; 0 del; 2 mod; 2206 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
8 lines changed: 6 ins; 0 del; 2 mod; 4935 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 590 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
5 lines changed: 5 ins; 0 del; 0 mod; 381 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
5 lines changed: 3 ins; 0 del; 2 mod; 762 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
4 lines changed: 3 ins; 0 del; 1 mod; 740 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
8 lines changed: 6 ins; 1 del; 1 mod; 1164 unchg

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

rev 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
36 lines changed: 30 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode 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 47935 : 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 47935 : 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 47935 : 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
10 lines changed: 7 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47935 : 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 47935 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
23 lines changed: 22 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode 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 47935 : 8176841: Additional Unicode Language-Tag Extensions
8189134: New system properties for the default Locale extensions
Reviewed-by:
39 lines changed: 12 ins; 26 del; 1 mod; 589 unchg

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

rev 47935 : 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 47935 : 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 47935 : 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
43 lines changed: 37 ins; 3 del; 3 mod; 509 unchg

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

rev 47935 : 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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 47935 : 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 47935 : 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
108 lines changed: 108 ins; 0 del; 0 mod; 0 unchg

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

rev 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
158 lines changed: 158 ins; 0 del; 0 mod; 0 unchg

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

rev 47935 : 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 47935 : 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 47935 : 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 47935 : 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 47937 : 8191349: Add a new method in j.t.f.DateTimeFormatter to reflect Unicode extensions
Reviewed-by:
853 lines changed: 853 ins; 0 del; 0 mod; 0 unchg

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

rev 47935 : 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 47935 : 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 47935 : 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 47935 : 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 47935 : 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 47936 : 8190918: Retrieve the region specific data regardless of language in locale
Reviewed-by:
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).