In DateTimeFormatter line 1508, this would be preferred: return new DateTimeFormatter(printerParser, locale, ds, resolverStyle, resolverFields, c, z);
In DateTimeFormatterBuilder.getLocalizedDateTimePattern() there is no spec change wrt using "rg". Should findRegionOverride() just return a Locale instead of am Optional<Locale>? It always seems to have an orElse(locale). Java-Time tests look good. thanks Stephen On 22 November 2017 at 19:04, Naoto Sato <naoto.s...@oracle.com> wrote: > I revised the proposed changes, including java.time changes suggested by > Stephen (CSR is still in progress): > > https://bugs.openjdk.java.net/browse/JDK-8191349 > > The entire webrev is located at: > > http://cr.openjdk.java.net/~naoto/8176841.8189134.8190918.8191349/webrev.05/ > > And the diff webrev from the last one is located at: > > http://cr.openjdk.java.net/~naoto/8191349/webrev.04-05/ > > I'd appreciate your reviews. > > Naoto > > > > On 11/9/17 3:34 PM, Naoto Sato wrote: >> >> Kindly requesting reviews. I incorporated a fix to the following issue >> raised by the test team: >> >> https://bugs.openjdk.java.net/browse/JDK-8190918 >> >> Here is the updated webrev: >> >> http://cr.openjdk.java.net/~naoto/8176841.8189134.8190918/webrev.04/ >> >> And the webrev since the one below (to address 8190918): >> >> http://cr.openjdk.java.net/~naoto/8190918/ >> >> Naoto >> >> >> >> On 11/2/17 2:42 PM, Naoto Sato wrote: >>> >>> Hello, >>> >>> Please review the proposed changes for the following issues: >>> >>> 8176841: Additional Unicode Language-Tag Extensions >>> 8189134: New system properties for the default Locale extensions >>> >>> The proposed changeset is located at: >>> >>> http://cr.openjdk.java.net/~naoto/8176841/webrev.03/ >>> >>> This serves as the implementation of JEP 314. >>> >>> Naoto >>> >>> >>> >