On Wed, 9 Sep 2026 12:07:28 GMT, Coleen Phillimore <[email protected]> wrote:
>> Coleen Phillimore has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - fix copyright >> - This is why I picked JVMTI_HEAP_REFERENCE_ARRAY_ELEMENT. It's what it >> used to return. > > For discussion sake, I needed to add a new ref kind JVMTI_REFERENCE_OTHER. I > was going to update the description of JVMTI_HEAP_REFERENCE_OTHER but it's > already sufficiently vague: > > <constant id="JVMTI_HEAP_REFERENCE_OTHER" num="27"> > Heap root reference: other heap root reference. > </constant> > > @sspitsyn The JVMTI GetLoadedClasses doesn't use the classes vector, it > walks the ClassLoaderData, and GetClassLoaderLoadedClasses walks the > dictionary inside the ClassLoaderData. > > So the compatibility issue with this would be an agent doing a switch > statement on JVMTI type for a class when starting from the > java.lang.ClassLoader (?) @coleenp Thank you for the update! It is nice for compatibility. ------------- PR Comment: https://git.openjdk.org/jdk/pull/32519#issuecomment-5609427162
