On Wed, 3 Jun 2026 14:45:45 GMT, Viktor Klang <[email protected]> wrote:
>> Chen Liang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Message updated
>
> src/java.base/share/classes/java/io/ObjectInputStream.java line 2249:
>
>> 2247: for (int i = 0; i < slots.length-1; i++) {
>> 2248: if (slots[i].hasData) {
>> 2249: new FieldValues(slots[i].desc, true);
>
> What's the performance impact of allocating the intermediate array?
The intermediate array is allocated in the mainline ObjectInputStream. I
reverted some code changes unrelated to the JEP 401 effort. And this is a
package-private method that returns a trusted array.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2326#discussion_r3349718216