Fixes for graphical corruption issue (processing#1490)

codeanticode · codeanticode · commit 73478da44fc2 · 2013-02-12T14:21:23.000-05:00
diff --git a/core/library/jogl-all.jar b/core/library/jogl-all.jar
diff --git a/core/src/processing/core/PApplet.java b/core/src/processing/core/PApplet.java
@@ -10214,6 +10214,7 @@ static public void runSketch(final String args[], final PApplet constructedApple
     }
 
     Frame frame = new Frame(displayDevice.getDefaultConfiguration());
+    frame.setBackground(new Color(0xCC, 0xCC, 0xCC)); // default Processing gray
 //    JFrame frame = new JFrame(displayDevice.getDefaultConfiguration());
       /*
       Frame frame = null;
diff --git a/core/src/processing/opengl/PGL.java b/core/src/processing/opengl/PGL.java
@@ -441,8 +441,6 @@ public class PGL {
   protected static FBObject.TextureAttachment backTexAttach;
   protected static FBObject.TextureAttachment frontTexAttach;
 
-  protected static boolean needToClearBuffers;
-
   ///////////////////////////////////////////////////////////
 
   // Texture rendering
@@ -538,7 +536,6 @@ public PGL(PGraphicsOpenGL pg) {
       fboLayerCreated = false;
       fboLayerInUse = false;
       firstFrame = false;
-      needToClearBuffers = false;
     }
 
     byteBuffer = allocateByteBuffer(1);
@@ -628,13 +625,12 @@ protected void initSurface(int antialias) {
     caps.setDepthBits(request_depth_bits);
     caps.setStencilBits(request_stencil_bits);
     caps.setAlphaBits(request_alpha_bits);
-
+    caps.setDefaultColor(pg.backgroundColor);
 
     if (toolkit == AWT) {
       canvasAWT = new GLCanvas(caps);
       canvasAWT.setBounds(0, 0, pg.width, pg.height);
-      canvasAWT.setBackground(Color.BLACK);
-      canvasAWT.setForeground(Color.BLACK);
+      canvasAWT.setBackground(new Color(pg.backgroundColor, true));
 
       pg.parent.setLayout(new BorderLayout());
       pg.parent.add(canvasAWT, BorderLayout.CENTER);
@@ -655,8 +651,7 @@ protected void initSurface(int antialias) {
       window = GLWindow.create(caps);
       canvasNEWT = new NewtCanvasAWT(window);
       canvasNEWT.setBounds(0, 0, pg.width, pg.height);
-      canvasNEWT.setBackground(Color.BLACK);
-      canvasNEWT.setForeground(Color.BLACK);
+      canvasNEWT.setBackground(new Color(pg.backgroundColor, true));
 
       pg.parent.setLayout(new BorderLayout());
       pg.parent.add(canvasNEWT, BorderLayout.CENTER);
@@ -684,7 +679,6 @@ protected void initSurface(int antialias) {
     fboLayerCreated = false;
     fboLayerInUse = false;
     firstFrame = true;
-    needToClearBuffers = true;
   }
 
 
@@ -701,7 +695,6 @@ protected void deleteSurface() {
     fboLayerCreated = false;
     fboLayerInUse = false;
     firstFrame = false;
-    needToClearBuffers = false;
   }
 
 
@@ -3137,17 +3130,6 @@ public void display(GLAutoDrawable glDrawable) {
         gl2x = null;
       }
 
-      if (needToClearBuffers) {
-        // Cleaning all buffers for the first time.
-        gl.glClearDepthf(1);
-        gl.glClearStencil(0);
-        gl.glClearColor(0, 0, 0, 0);
-        gl.glClear(GL.GL_COLOR_BUFFER_BIT | GL.GL_DEPTH_BUFFER_BIT |
-                   GL.GL_STENCIL_BUFFER_BIT);
-        needToClearBuffers = false;
-        drawable.swapBuffers();
-      }
-
       if (USE_JOGL_FBOLAYER && capabilities.isFBO()) {
         // The onscreen drawing surface is backed by an FBO layer.
         GLFBODrawable fboDrawable = null;

Original file line number	Diff line number	Diff line change
`@@ -10214,6 +10214,7 @@ static public void runSketch(final String args[], final PApplet constructedApple`
`10214`	`10214`	`}`
`10215`	`10215`
`10216`	`10216`	`Frame frame = new Frame(displayDevice.getDefaultConfiguration());`
	`10217`	`+ frame.setBackground(new Color(0xCC, 0xCC, 0xCC)); // default Processing gray`
`10217`	`10218`	`// JFrame frame = new JFrame(displayDevice.getDefaultConfiguration());`
`10218`	`10219`	`/*`
`10219`	`10220`	`Frame frame = null;`