On Fri, 19 Sep 2025 03:06:42 GMT, Ioi Lam <[email protected]> wrote: >> Simplify the boilerplate code in jvm.cpp that calls >> `JvmtiThreadState::class_to_verify_considering_redefinition()`, and reduce >> the number of `InstanceKlass::cast()` calls. >> >> I also changed a few fields/arguments from `Klass*` to `InstanceKlass*` as >> these are used exclusively with `InstanceKlass*`. > > Ioi Lam has updated the pull request with a new target base due to a merge or > a rebase. The incremental webrev excludes the unrelated changes brought in by > the merge/rebase. The pull request contains three additional commits since > the last revision: > > - @coleenp and @dholmes-ora comments -- simplify implementation; fixed code > formatting > - Merge branch 'master' into > 8367719-refactor-jni-code-that-uses-class_to_verify_considering_redefinition > - 8367719: Refactor JNI code that uses > class_to_verify_considering_redefinition()
Looks good. ------------- Marked as reviewed by coleenp (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/27303#pullrequestreview-3244497323
