On Wed, 30 Oct 2024 18:08:16 GMT, Ioi Lam <ik...@openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Change InternalErrors to asserts
>
> src/java.base/share/classes/java/lang/Integer.java line 966:
> 
>> 964:                 // Otherwise, the identity checks between archived 
>> Integers and
>> 965:                 // runtime-cached Integers would fail.
>> 966:                 int archivedIdx = (archivedCache == null) ? 0 : 
>> archivedCache.length;
> 
> I think the name of `archiveIdx` isn't clear. How about `archivedSize`?

Yes, we can do `archivedSize`. Done in new commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21737#discussion_r1824010416

Reply via email to