<i18n dev> RFR: 8317742: ISO Standard Date Format implementation consistency on DateTimeFormatter and String.format [v8]

[Naoto Sato]

On Fri, 17 Nov 2023 22:16:36 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> 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).

Haven't tried, but I think `t.getChronology() instanceof IsoChronology` should work

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16033#discussion_r1397972898