On Wed, 15 Apr 2026 17:34:01 GMT, Shiv Shah <[email protected]> wrote:
>> Fixed three test files that fail with -Xcheck:jni on debug builds: >> >> popframe007.cpp: CallStaticVoidMethod in native B() was called with the >> declaring class (popframe007) instead of TestThread where the method ID >> belongs. Stored TestThread class as a global ref in getReady() and used it >> in B() >> >> aod.cpp: nsk_aod_agentLoaded() and nsk_aod_agentFinished() call >> CallStaticVoidMethod without checking for pending exceptions. Added >> nsk_aod_checkJNIException wrapper that describes, clears, and fails on >> unexpected exceptions. Fixes all 13 AttachOnDemand tests >> >> retransform003.cpp: ClassFileLoadHook callback calls CallObjectMethod and >> CallStaticObjectMethod without checking for pending exceptions. Added >> ExceptionCheck/ExceptionClear after both calls >> >> bi02t001 and bi03t001 pass without changes, NMT corruption issue from JDK 22 >> appears resolved on JDK 27 >> >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Shiv Shah has updated the pull request incrementally with one additional > commit since the last revision: > > Remove unnecessary return check per review @shivshah-oracle Your change (at version f5b316026feb91122eaf07bf0c0341fa3eb10daf) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30711#issuecomment-4261567467
