On Fri, 9 Jul 2021 12:20:53 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

> Alternatively, I could add a new JNI function to check the version, and call 
> that right after the `loadLibrary`. That might be cleaner than doing it as 
> part of `initIDs`

I think it's ok to keep it in the initIDs, as that is called immediately after 
the loadLibrary. A new JNI function seems a bit overkill -- it would have fit 
nicely in the JNI_OnLoad but I do agree that that is not the best approach, 
since it will convert the exception/error into an UnsatiffiedLinkError.
Hence, +1 on your current approach with doing it in initIDs.

-------------

PR: https://git.openjdk.java.net/jfx/pull/567

Reply via email to