GL/GLContext: Properly define swapInterval incl. default value for EGL (1) and desktop (undefined).

*GLContext.setSwapIntervalImpl: Simple return success, set state in GLContext.

1 files changed, 7 insertions, 6 deletions

diff --git a/src/jogl/classes/jogamp/opengl/egl/EGLContext.java b/src/jogl/classes/jogamp/opengl/egl/EGLContext.java
index dc47799c1..f5f9f62c4 100644
--- a/src/jogl/classes/jogamp/opengl/egl/EGLContext.java
+++ b/src/jogl/classes/jogamp/opengl/egl/EGLContext.java
@@ -274,18 +274,19 @@ public abstract class EGLContext extends GLContextImpl {
         return sb;
     }
 
-    protected void setSwapIntervalImpl(int interval) {
+    @Override
+    protected boolean setSwapIntervalImpl(int interval) {
         // FIXME !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
         // eglSwapInterval(..) issued:
         //   Android 4.0.3 / Pandaboard ES / PowerVR SGX 540: crashes
         // FIXME !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-        if( ! ( Platform.OSType.ANDROID == Platform.getOSType() && getGLRendererString(true).contains("powervr") ) ) {
-            if (EGL.eglSwapInterval(((EGLDrawable)drawable).getDisplay(), interval)) {
-                currentSwapInterval = interval ;
+        if( Platform.OSType.ANDROID == Platform.getOSType() && getGLRendererString(true).contains("powervr") ) {
+            if(DEBUG) {
+                System.err.println("Ignored: eglSwapInterval("+interval+") - cause: OS "+Platform.getOSType() + " / Renderer " + getGLRendererString(false));
             }
-        } else if(DEBUG) {
-            System.err.println("Ignored: eglSwapInterval("+interval+") - cause: OS "+Platform.getOSType() + " / Renderer " + getGLRendererString(false));
+            return false;
         }
+        return EGL.eglSwapInterval(((EGLDrawable)drawable).getDisplay(), interval);
     }
 
     public abstract void bindPbufferToTexture();