> `Instant.parse()` is expected to use the offset zone pattern `+HH:mm:ss` (as > defined by `DateTimeFormatterBuilder.appendOffsetId()`), but it fails to > parse hour-only offsets such as `+02`. This is because the actual > implementation uses `+HH:MM:ss` as the pattern. While replacing the pattern > in the implementation as with the specification would allow hour-only > offsets, it would also introduce compatibility issues, i.e., printing would > omit the minutes field when it is zero. So, it is preferable to update the > specification to match the implementation. A CSR has also been drafted for > this change.
Naoto Sato has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains nine additional commits since the last revision: - DateTimeFormatterBuilder wording - Added non-zero offset test cases - Merge branch 'master' into JDK-8364752-Instant-ISO8601 - Update src/java.base/share/classes/java/time/format/DateTimeFormatter.java Right. Changing to your suggested wording Co-authored-by: Roger Riggs <roger.ri...@oracle.com> - copyright year update - test cases - allow all ISO 8601 offsets - allow hour-only offsets - initial commit ------------- Changes: - all: https://git.openjdk.org/jdk/pull/26708/files - new: https://git.openjdk.org/jdk/pull/26708/files/8bc222af..0d78a520 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26708&range=06 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26708&range=05-06 Stats: 16207 lines in 511 files changed: 8985 ins; 5233 del; 1989 mod Patch: https://git.openjdk.org/jdk/pull/26708.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/26708/head:pull/26708 PR: https://git.openjdk.org/jdk/pull/26708
