RFR: 8254569: Remove hard dependency on Dispman in Monocle fb rendering [v4]

[Johan Vos]

On Thu, 5 Nov 2020 02:48:17 GMT, John Neffenger <github.com+1413266+jgneff at openjdk.org> wrote:

>> Johan Vos has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   add one more @override to process reviewer comments
>
> modules/javafx.graphics/src/main/native-glass/monocle/egl/egl_ext.h line 42:
> 
>> 40:      jlong readSurface, jlong eglContext);
>> 41: 
>> 42: extern jboolean doEglSwapBuffers(jlong eglDisplay, jlong eglSurface);
> 
> Is there a reason for using different types for the same variables in the method definitions?
> 
> * The *window handle* is `long` and `jlong`.
> * The *display handle* is `long`, `void*`, and `jlong`.
> * The others (*config*, *surface*, and *context*) are all `jlong`.

Initially, I wanted to avoid jni.h in this external file. However, since 2 of the functions return a boolean value, I introduced jboolean, but didn't change all values. 
We can still go back to the non-jni types, e.g. by having an int instead of a jboolean, but at this moment I think the presence of jni.h should not be a showstopper. 
Implementations of this interface will be (or bundled with) code that does not know about Java, hence I tried to avoid jni types, but in case jni.h is not available, the implementation can easily typedef the required types.

I made it consistent now to use jlong instead of long.

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

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