On Fri, 10 Nov 2023 03:31:26 GMT, Shaojin Wen <d...@openjdk.org> wrote:
>> j.u.Formatter now prints "%tF" (iso standard date) and the result is
>> incorrect when processing year < 0 or year > 9999
>
> Shaojin Wen has updated the pull request incrementally with one additional
> commit since the last revision:
>
> fix format 'ja-JP-u-ca-japanese' ChronoLocalDate
src/java.base/share/classes/java/util/Formatter.java line 4507:
> 4505: || t instanceof LocalDateTime
> 4506: || t instanceof LocalDate) {
> 4507: yearField = ChronoField.YEAR;
This enumeration of type doesn't seem very principled and may not be correct
for an arbitrary Temporal.
(But so far, I don't have a better suggestion).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16033#discussion_r1397946102
