On Fri, 12 Feb 2021 04:48:27 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> wrote:
>> The various CF*Copy* functions called here need a matching CFRelease > > src/java.desktop/macosx/native/libawt_lwawt/font/AWTFont.m line 578: > >> 576: (*env)->CallBooleanMethod(env, arrayListOfString, addMID, >> jFontName); >> 577: if ((*env)->ExceptionOccurred(env)) { >> 578: CFRelease(fds); > > but DeleteLocalRef(jFontName) shouldn't be called here too? Unnecessary. The VM releases it when we return right here (next line) The other delete is needed because we are in a loop and we don't know how many we might need and it could exceed what the VM provisions for. ------------- PR: https://git.openjdk.java.net/jdk/pull/2532