On Wed, 1 Jul 2026 18:52:56 GMT, Chris Plummer <[email protected]> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   review: tweaked spec clarification
>
> src/hotspot/share/prims/jvmti.xml line 14077:
> 
>> 14075:             JNI local reference to the object that was allocated.
>> 14076:             Null when preview features are enabled, the capability 
>> <code>can_support_value_objects</code>
>> 14077:             is enabled and the allocated object has no identity.
> 
> Suggestion:
> 
>             is enabled, and the allocated object has no identity.

Thanks. Fixed now.

> src/hotspot/share/prims/jvmti.xml line 14154:
> 
>> 14152:           JNI local reference to the object that was allocated.
>> 14153:           Null when preview features are enabled, the capability 
>> <code>can_support_value_objects</code>
>> 14154:           is enabled and the allocated object has no identity.
> 
> Suggestion:
> 
>           is enabled, and the allocated object has no identity.

Thanks. Fixed now.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2607#discussion_r3508900437
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2607#discussion_r3508901604

Reply via email to