On Wed, 14 Jan 2026 04:21:10 GMT, Chris Plummer <[email protected]> wrote:

>> src/java.se/share/data/jdwp/jdwp.spec line 1830:
>> 
>>> 1828:             (Error VM_DEAD)
>>> 1829:         )
>>> 1830:     )
>> 
>> I wonder if the `(Error NOT_IMPLEMENTED)` should also be included. Should we 
>> also consider to introduce one more field in the `CapabilitiesNew=17` 
>> command?
>
> We didn't do similar for Loom related APIs. I looked through the current 
> canXXX list, and they all seem related to JVMTI capabilities, which don't 
> apply to these two APIs. I think the expectation should be that the user 
> checks the JDWP version to determine if it is implemented.

I see now how the "versioning" works.
I added `boolean VirtualMachineImpl.canUseIsSameObject()` to check if 
IsSameObject/ObjectHashCode are available

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/1834#discussion_r2692317354

Reply via email to