Code Review for open

Prepared by:naoto on Mon Apr 23 15:34:28 PDT 2018
Workspace:/projects/jdk/11/tz/open
Compare against: http://hg.openjdk.java.net/jdk/jdk
Compare against version:qparent
Summary of changes: 702 lines changed: 547 ins; 63 del; 92 mod; 21199 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/Bundle.java 

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
36 lines changed: 5 ins; 27 del; 4 mod; 738 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
39 lines changed: 27 ins; 5 del; 7 mod; 1034 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
38 lines changed: 26 ins; 0 del; 12 mod; 952 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
11 lines changed: 9 ins; 0 del; 2 mod; 118 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
19 lines changed: 7 ins; 0 del; 12 mod; 361 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
8 lines changed: 7 ins; 0 del; 1 mod; 2512 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
3 lines changed: 0 ins; 0 del; 3 mod; 3472 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
23 lines changed: 19 ins; 0 del; 4 mod; 250 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
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 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
5 lines changed: 1 ins; 0 del; 4 mod; 56 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
33 lines changed: 24 ins; 0 del; 9 mod; 508 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
28 lines changed: 0 ins; 20 del; 8 mod; 132 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
10 lines changed: 7 ins; 1 del; 2 mod; 278 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
6 lines changed: 0 ins; 5 del; 1 mod; 342 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
14 lines changed: 5 ins; 4 del; 5 mod; 119 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/TimeZone/Bug8149452.java 

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
18 lines changed: 14 ins; 1 del; 3 mod; 57 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/TimeZone/CLDRDisplayNamesTest.java 

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
8 lines changed: 0 ins; 0 del; 8 mod; 116 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/util/TimeZone/TimeZoneTest.java 

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
3 lines changed: 1 ins; 0 del; 2 mod; 735 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
1 line changed: 0 ins; 0 del; 1 mod; 8380 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
2 lines changed: 0 ins; 0 del; 2 mod; 490 unchg

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

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
272 lines changed: 272 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/sun/util/resources/cldr/TimeZoneNamesTest.java 

rev 49854 : 8181157: CLDR Timezone name fallback implementation
Reviewed-by: sherman
123 lines changed: 123 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).