On Tue, 16 Sep 2025 15:57:48 GMT, Coleen Phillimore <[email protected]> wrote:
>> 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() > > src/hotspot/share/prims/jvm.cpp line 2291: > >> 2289: >> 2290: JVM_ENTRY(const char*, JVM_GetClassNameUTF(JNIEnv *env, jclass cls)) >> 2291: Klass* k = get_klass_considering_redefinition(cls, thread); > > Even if it's a redefined class, it'll have the same name so this isn't > necessary. I usually don't want to make semantic changes in a clean-up PR, but this one seems pretty safe so I changed it anyway. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27303#discussion_r2361635458
