+1. That's nicer.
-Joe
On 2/11/20 10:17 AM, naoto.s...@oracle.com wrote:
Hi,
I modified the proposed changeset. Removed the hard coded 3-letter id
compatibility mappings (oldMappings) from CLDRConverter.java. Instead,
using public ZoneId.SHORT_IDS that contain the same set of mappings
(wasn't aware it was defined in the spec!)
Here is the updated webrev:
http://cr.openjdk.java.net/~naoto/8234347.8236548/webrev.01/
Naoto
On 2/7/20 1:44 PM, naoto.s...@oracle.com wrote:
Hello,
Please review the fix to the following issues:
https://bugs.openjdk.java.net/browse/JDK-8234347
https://bugs.openjdk.java.net/browse/JDK-8236548
The proposed changeset is located at:
http://cr.openjdk.java.net/~naoto/8234347.8236548/webrev.00/
This changeset includes the following changes:
- English time zone names missing in CLDR source files are no longer
copied from COMPAT provider at build time. Rather it is synthesized
at runtime, which has been the way other locales did.
- Runtime CLDR time zone name provider fallback logic has been
refined. It now falls back to parent locales per each missing name,
instead of resource bundle. Also, region fall back is now using
exemplar city to synthesize the name (e.g., "Turkey" meta zone)
- Minor fix in DateTimeFormatterBuilder on zone text parsing. It now
parses zone texts that start with "UTC", yet it is ZoneId.
Naoto
