GLContext GLDebugMessages: Add synchronous status/dumpStack; Remove length in aliased glDebugMessageInsert.

- GLDebugMessages add synchronous status - defaults to true
- GLContext/GLDebugMessages add dumpStack() if jogl.debug.DebugGL is set
- Remove param length in aliased glDebugMessageInsert.

3 files changed, 72 insertions, 26 deletions

diff --git a/src/jogl/classes/javax/media/opengl/GLContext.java b/src/jogl/classes/javax/media/opengl/GLContext.java
index 08bbcbb96..30cc9c2ea 100644
--- a/src/jogl/classes/javax/media/opengl/GLContext.java
+++ b/src/jogl/classes/javax/media/opengl/GLContext.java
@@ -539,6 +539,21 @@ public abstract class GLContext {
   public abstract String getGLDebugMessageExtension();
 
   /**
+   * @return the current synchronous debug behavior via
+   * @see #setSynchronous(boolean)
+   */
+  public abstract boolean isGLDebugSynchronous();
+    
+  /**
+   * Enables or disables the synchronous debug behavior via
+   * {@link GL2GL3#GL_DEBUG_OUTPUT_SYNCHRONOUS_ARB glEnable/glDisable(GL_DEBUG_OUTPUT_SYNCHRONOUS_ARB)}, 
+   * if extension is {@link #GL_ARB_debug_output}.
+   * There is no equivalent for {@link #GL_AMD_debug_output}.
+   * <p> The default is <code>true</code>, ie {@link GL2GL3#GL_DEBUG_OUTPUT_SYNCHRONOUS_ARB}.</p> 
+   */
+  public abstract void setGLDebugSynchronous(boolean synchronous);
+  
+  /**
    * @return true if the GLDebugOutput feature is enabled or not.
    */
   public abstract boolean isGLDebugMessageEnabled();
@@ -586,12 +601,6 @@ public abstract class GLContext {
   public abstract void removeGLDebugListener(GLDebugListener listener);
   
   /**
-   * @return number of added {@link GLDebugListener}. If > 0, the GLDebugFeature is turned on, otherwise off. 
-   * @see #enableGLDebugMessage(boolean) 
-   */
-  public abstract int getGLDebugListenerSize();
-  
-  /**
    * Generic entry for {@link GL2GL3#glDebugMessageControlARB(int, int, int, int, IntBuffer, boolean)} 
    * and {@link GL2GL3#glDebugMessageEnableAMD(int, int, int, IntBuffer, boolean)} of the GLDebugOutput feature.
    * @see #enableGLDebugMessage(boolean) 
@@ -610,7 +619,7 @@ public abstract class GLContext {
    * and {@link GL2GL3#glDebugMessageInsertAMD(int, int, int, int, String)} of the GLDebugOutput feature.
    * @see #enableGLDebugMessage(boolean) 
    */
-  public abstract void glDebugMessageInsert(int source, int type, int id, int severity, int length, String buf);
+  public abstract void glDebugMessageInsert(int source, int type, int id, int severity, String buf);
   
   public static final int GL_VERSIONS[][] = {
       /* 0.*/ { -1 },
diff --git a/src/jogl/classes/jogamp/opengl/GLContextImpl.java b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
index 6128b30b4..1d5c6911b 100644
--- a/src/jogl/classes/jogamp/opengl/GLContextImpl.java
+++ b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
@@ -407,7 +407,7 @@ public abstract class GLContextImpl extends GLContext {
         
         if(DEBUG_GL) {
             gl = gl.getContext().setGL( GLPipelineFactory.create("javax.media.opengl.Debug", null, gl, null) );
-            glDebugHandler.addListener(new GLDebugMessageHandler.StdErrGLDebugListener());
+            glDebugHandler.addListener(new GLDebugMessageHandler.StdErrGLDebugListener(true));
         }
         if(TRACE_GL) {
             gl = gl.getContext().setGL( GLPipelineFactory.create("javax.media.opengl.Trace", null, gl, new Object[] { System.err } ) );
@@ -1096,17 +1096,23 @@ public abstract class GLContextImpl extends GLContext {
       return glDebugHandler.isEnabled();
   }
   
-  public int getContextCreationFlags() {
+  public final int getContextCreationFlags() {
       return additionalCtxCreationFlags;                
   }
 
-  public void setContextCreationFlags(int flags) {
+  public final void setContextCreationFlags(int flags) {
       if(!isCreated()) {
           additionalCtxCreationFlags = flags & GLContext.CTX_OPTION_DEBUG;
       }
   }
   
-  public void enableGLDebugMessage(boolean enable) throws GLException {
+  public final boolean isGLDebugSynchronous() { return glDebugHandler.isSynchronous(); }     
+  
+  public final void setGLDebugSynchronous(boolean synchronous) {
+      glDebugHandler.setSynchronous(synchronous);
+  }
+  
+  public final void enableGLDebugMessage(boolean enable) throws GLException {
       if(!isCreated()) {
           if(enable) {
               additionalCtxCreationFlags |=  GLContext.CTX_OPTION_DEBUG;
@@ -1119,19 +1125,15 @@ public abstract class GLContextImpl extends GLContext {
       }
   }
   
-  public void addGLDebugListener(GLDebugListener listener) { 
+  public final void addGLDebugListener(GLDebugListener listener) { 
       glDebugHandler.addListener(listener);
   }
   
-  public void removeGLDebugListener(GLDebugListener listener) {
+  public final void removeGLDebugListener(GLDebugListener listener) {
       glDebugHandler.removeListener(listener);
   }    
   
-  public int getGLDebugListenerSize() {
-      return glDebugHandler.listenerSize();      
-  }
-  
-  public void glDebugMessageControl(int source, int type, int severity, int count, IntBuffer ids, boolean enabled) {
+  public final void glDebugMessageControl(int source, int type, int severity, int count, IntBuffer ids, boolean enabled) {
       if(glDebugHandler.isExtensionARB()) {
           gl.getGL2GL3().glDebugMessageControlARB(source, type, severity, count, ids, enabled);
       } else if(glDebugHandler.isExtensionAMD()) {
@@ -1139,7 +1141,7 @@ public abstract class GLContextImpl extends GLContext {
       }      
   }
   
-  public void glDebugMessageControl(int source, int type, int severity, int count, int[] ids, int ids_offset, boolean enabled) {
+  public final void glDebugMessageControl(int source, int type, int severity, int count, int[] ids, int ids_offset, boolean enabled) {
       if(glDebugHandler.isExtensionARB()) {
           gl.getGL2GL3().glDebugMessageControlARB(source, type, severity, count, ids, ids_offset, enabled);
       } else if(glDebugHandler.isExtensionAMD()) {
@@ -1147,12 +1149,11 @@ public abstract class GLContextImpl extends GLContext {
       }
   }
   
-  public void glDebugMessageInsert(int source, int type, int id, int severity, int length, String buf) {
+  public final void glDebugMessageInsert(int source, int type, int id, int severity, String buf) {
       if(glDebugHandler.isExtensionARB()) {
-          gl.getGL2GL3().glDebugMessageInsertARB(source, type, id, severity, length, buf);
+          gl.getGL2GL3().glDebugMessageInsertARB(source, type, id, severity, -1, buf);
       } else if(glDebugHandler.isExtensionAMD()) {
-          if(0>length) { length = 0; }
-          gl.getGL2GL3().glDebugMessageInsertAMD(GLDebugMessage.translateARB2AMDCategory(source, type), severity, id, length, buf);
+          gl.getGL2GL3().glDebugMessageInsertAMD(GLDebugMessage.translateARB2AMDCategory(source, type), severity, id, 0, buf);
       }      
   }
 }
diff --git a/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java b/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java
index f67c916ea..09be202dd 100644
--- a/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java
+++ b/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java
@@ -78,6 +78,7 @@ public class GLDebugMessageHandler {
     private int extType;
     private long glDebugMessageCallbackProcAddress;
     private boolean extAvailable; 
+    private boolean synchronous;
     
     // licefycle: enable - disable/EOL
     private long handle;
@@ -94,6 +95,7 @@ public class GLDebugMessageHandler {
         this.extType = 0;
         this.extAvailable = false; 
         this.handle = 0;
+        this.synchronous = true;
     }
     
     public void init(boolean enable) {
@@ -176,7 +178,32 @@ public class GLDebugMessageHandler {
     }
     
     /**
-     * @throws GLException if context not current or callback registration failed (enable) 
+     * @see javax.media.opengl.GLContext#isGLDebugSynchronous() 
+     */
+    public final boolean isSynchronous() { return synchronous; }
+    
+    /**
+     * @see javax.media.opengl.GLContext#setGLDebugSynchronous(boolean) 
+     */
+    public final void setSynchronous(boolean synchronous) {
+        this.synchronous = synchronous;
+        if(isEnabled() && isExtensionARB()) {
+            setSynchronousImpl();
+        }
+    }    
+    private final void setSynchronousImpl() {
+        if(synchronous) {
+            ctx.getGL().glEnable(GL2GL3.GL_DEBUG_OUTPUT_SYNCHRONOUS_ARB);
+        } else {
+            ctx.getGL().glDisable(GL2GL3.GL_DEBUG_OUTPUT_SYNCHRONOUS_ARB);
+        }        
+        if(DEBUG) {
+            System.err.println("GLDebugMessageHandler: synchronous "+synchronous);
+        }
+    }
+    
+    /**
+     * @see javax.media.opengl.GLContext#enableGLDebugMessage(boolean) 
      */
     public final void enable(boolean enable) throws GLException {
         ctx.validateCurrent();
@@ -186,6 +213,7 @@ public class GLDebugMessageHandler {
         enableImpl(enable);
     }        
     final void enableImpl(boolean enable) throws GLException {
+        setSynchronousImpl();
         if(enable) {
             if(0 == handle) {
                 handle = register0(glDebugMessageCallbackProcAddress, extType);
@@ -234,9 +262,17 @@ public class GLDebugMessageHandler {
         }
     }
     
-    public static class StdErrGLDebugListener implements GLDebugListener {        
+    public static class StdErrGLDebugListener implements GLDebugListener {
+        boolean threadDump;
+        
+        public StdErrGLDebugListener(boolean threadDump) {
+            this.threadDump = threadDump;
+        }
         public void messageSent(GLDebugMessage event) {
-            System.err.println(event);            
+            System.err.println(event);
+            if(threadDump) {
+                Thread.dumpStack();
+            }
         }        
     }