Fixed Issue 151: starting up the Animator before the GLJPanel has been shown result in an error

The root cause of this error was the fact that
WindowsPbufferGLContext.destroyImpl() uses WGL extensions to clean up
resources associated with the pbuffer. Because these extensions are in
the public WGL interface, they are wrapped by the DebugGL. However, an
OpenGL context is not current at the time these routines are called,
and it is illegal to call glGetError() at those points. The DebugGL
pipeline was implicitly calling glGetError() after each of those
calls, leading to the failure.

This bug unmasked a couple of others. The code in the DebugGL needed a
recursion count to make sure that glGetError() didn't get called in an
infinite loop. Also, as a side effect of the fix for Issue 160,
calling getGL() on the GLJPanel outside of GLEventListener.init() was
causing a NullPointerException to be thrown. The GLJPanel has been
fixed to return null in this case, and the specification of
GLDrawable.getGL() has been improved. In order to make the behavior
between the GLCanvas and GLJPanel similar, the GL object is now reset
in the GLDrawable each time the underlying OpenGL context is
recreated. This allows end users to set up e.g. the DebugGL
unconditionally in their GLEventListener.init() method. The JOGL demos
have been changed to reflect this.

The test case in the bug report will be updated with code similar to
the originally submitted test case (i.e., the Animator is started
early) but which now works.


git-svn-id: file:///usr/local/projects/SUN/JOGL/git-svn/../svn-server-sync/jogl-demos/trunk@71 3298f667-5e0e-4b4a-8ed4-a3559d26a5f4

1 files changed, 4 insertions, 12 deletions

diff --git a/src/demos/hwShadowmapsSimple/HWShadowmapsSimple.java b/src/demos/hwShadowmapsSimple/HWShadowmapsSimple.java
index 13f3997..f97e875 100644
--- a/src/demos/hwShadowmapsSimple/HWShadowmapsSimple.java
+++ b/src/demos/hwShadowmapsSimple/HWShadowmapsSimple.java
@@ -161,12 +161,8 @@ public class HWShadowmapsSimple {
   class Listener implements GLEventListener {
 
     public void init(GLDrawable drawable) {
-      // init() might get called more than once if the GLCanvas is
-      // added and removed, but we only want to install the DebugGL
-      // pipeline once
-      //      if (!(drawable.getGL() instanceof DebugGL)) {
-      //        drawable.setGL(new DebugGL(drawable.getGL()));
-      //      }
+      // Use debug pipeline
+      // drawable.setGL(new DebugGL(drawable.getGL()));
 
       GL gl = drawable.getGL();
       GLU glu = drawable.getGLU();
@@ -378,12 +374,8 @@ public class HWShadowmapsSimple {
 
   class PbufferListener implements GLEventListener {
     public void init(GLDrawable drawable) {
-      // init() might get called more than once if the GLCanvas is
-      // added and removed, but we only want to install the DebugGL
-      // pipeline once
-      //      if (!(drawable.getGL() instanceof DebugGL)) {
-      //        drawable.setGL(new DebugGL(drawable.getGL()));
-      //      }
+      // Use debug pipeline
+      // drawable.setGL(new DebugGL(drawable.getGL()));
 
       GL gl = drawable.getGL();
       GLU glu = drawable.getGLU();