On Wed, 19 Apr 2023 06:50:45 GMT, Peter Hofer <pho...@openjdk.org> wrote:
>> src/java.base/share/native/libjava/jni_util.c line 757: >> >>> 755: jnuEncoding = (*env)->NewGlobalRef(env, charset.l); >>> 756: (*env)->DeleteLocalRef(env, charset.l); >>> 757: break; >> >> Could return immediately > > No, then the method id cache would not get initialized below. But I will add > a comment. You are right, those init code should follow. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13499#discussion_r1171618399
