On Thu, 2 Nov 2023 17:45:51 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Amazingly enough I seem to have the JNI straight on the first try. > >> Amazingly enough I seem to have the JNI straight on the first try. > > Oh, I wasn't expecting that. My comment was suggesting we change > LauncherHelper.checkAndLoadMain to set both appClass and mainType after the > validate succeeds. I thought it was a clearer way to get rid of the side effect. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16461#discussion_r1380585433
