On Wed, 24 Nov 2021 23:25:22 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> This fix intends to honor the type (std/dst/generic) of parsed zone name for >> selecting the offset at the overlap. Corresponding CSR has also been drafted. > > Naoto Sato has updated the pull request incrementally with two additional > commits since the last revision: > > - Replaced integer literals. > - Refined wording #2 src/java.base/share/classes/java/time/format/DateTimeFormatter.java line 508: > 506: * of {@link ChronoLocalDateTime#atZone(ZoneId)}. If the {@code ZoneId} > was > 507: * parsed from the zone name that does not indicate daylight saving > time, then > 508: * the standard offset will be used at the local time-line overlap. As written, I would read it as being the generic zone name that gets altered, rather than a zone name that explicitly indicates "winter" time. maybe: "If the {@code ZoneId} was parsed from a zone name that indicates whether daylight saving time in in operation or not, then that fact will be used to select the correct offset at the local time-line overlap." src/java.base/share/classes/java/time/format/DateTimeFormatterBuilder.java line 4906: > 4904: private static class LENIENT extends CI { > 4905: > 4906: private LENIENT(String k, String v, int t, PrefixTree > child) { Is class `LENIENT` actually in use? ------------- PR: https://git.openjdk.java.net/jdk/pull/6527