On Mon, 5 Jun 2023 20:55:30 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8304290: Some JNI calls made without checking exceptions in media [v2] > > modules/javafx.media/src/main/native/jfxmedia/jni/JavaPlayerEventDispatcher.cpp > line 199: > >> 197: } >> 198: >> 199: pEnv->DeleteLocalRef(jmessage); > > This needs to be moved inside the previous `if` block. Fixed. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1094#discussion_r1220513572
