On Fri, 27 Mar 2026 00:12:49 GMT, EunHyunsu <[email protected]> wrote:
>> I know this is pre-existing logic but would it be hard to use `java.time` >> APIs instead of `Calendar`? >> Could that method return an `Instant` instead of a `Calendar`? > > I looked into it, but `DateTimeFormatter` performs a day-of-week cross-check > that `SimpleDateFormat` silently ignores, which would change observable > parsing behavior and affect existing tests. > > I think the migration would be cleaner as a separate follow-up PR. Would that > be acceptable? Yes. What prompted my suggestion is the use of Calendar being split-out in two methods now rather than being confined to a single one. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30341#discussion_r3000233567
