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

Reply via email to