On Wed, 26 Feb 2025 19:48:10 GMT, Justin Lu <j...@openjdk.org> wrote:

>> Please review this PR which prevents an `AIOOBE` from leaking out when 
>> `java.util.Calendar.Builder` is used to build a Japanese calendar with an 
>> era value too large.
>> 
>> Note that we don't check under `BEFORE_MEIJI`/0 as historically Japanese 
>> calendar ignores negative values during normalization. See 
>> `JapaneseImperialCalendar` L2018: `date.setEra(era > 0 ? eras[era] : 
>> null);`. 
>> 
>> We also check against `eras.length` over `REIWA`/5 due to the possibility of 
>> additional eras via the property override or future eras in general.
>
> Justin Lu has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Replace 7 w/ more meaningful value

Thanks. I reviewed the CSR as well.

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

Marked as reviewed by naoto (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23789#pullrequestreview-2645746475

Reply via email to