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

>> src/jdk.jdi/share/classes/com/sun/tools/jdi/VirtualMachineImpl.java line 841:
>> 
>>> 839:         return versionInfo().jdwpMajor >= 27;
>>> 840:     }
>>> 841: 
>> 
>> Nit: We may want to generalize this with something like 
>> 'maySupportValueClasses()'.
>
> I'm not so sure this API is really even needed. I don't believe the current 
> calls to it are valid.

version info is private API in VirtualMachineImpl.
replaced with `supportsValueClasses` method

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

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

Reply via email to