From df85f0dfafc09e147f9d422adf5ee8eabf67977b Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Tue, 27 Sep 2011 11:59:06 +0200 Subject: Adapt to GlueGen's Lock ChangeSet: e4baba27507ce78e64a150ec6f69fb96f5721a34 ; Use generics --- .../com/jogamp/opengl/util/AWTAnimatorImpl.java | 26 +++++++++++----------- .../com/jogamp/opengl/util/AnimatorBase.java | 8 +++---- .../jogamp/opengl/util/DefaultAnimatorImpl.java | 4 ++-- .../classes/javax/media/opengl/awt/GLCanvas.java | 17 +++++++++----- src/jogl/classes/jogamp/opengl/GLPbufferImpl.java | 7 ++---- 5 files changed, 32 insertions(+), 30 deletions(-) (limited to 'src/jogl') diff --git a/src/jogl/classes/com/jogamp/opengl/util/AWTAnimatorImpl.java b/src/jogl/classes/com/jogamp/opengl/util/AWTAnimatorImpl.java index 9c77a0508..93b75e70b 100644 --- a/src/jogl/classes/com/jogamp/opengl/util/AWTAnimatorImpl.java +++ b/src/jogl/classes/com/jogamp/opengl/util/AWTAnimatorImpl.java @@ -54,21 +54,21 @@ import javax.media.opengl.GLAutoDrawable; class AWTAnimatorImpl implements AnimatorBase.AnimatorImpl { // For efficient rendering of Swing components, in particular when // they overlap one another - private List lightweights = new ArrayList(); - private Map repaintManagers = new IdentityHashMap(); - private Map dirtyRegions = new IdentityHashMap(); + private List lightweights = new ArrayList(); + private Map repaintManagers = new IdentityHashMap(); + private Map dirtyRegions = new IdentityHashMap(); - public void display(ArrayList drawables, + public void display(ArrayList drawables, boolean ignoreExceptions, boolean printExceptions) { for (int i=0; i iter = lightweights.iterator(); iter.hasNext(); ) { + JComponent comp = iter.next(); RepaintManager rm = RepaintManager.currentManager(comp); rm.markCompletelyDirty(comp); repaintManagers.put(rm, rm); @@ -148,16 +148,16 @@ class AWTAnimatorImpl implements AnimatorBase.AnimatorImpl { } // Dirty any needed regions on non-optimizable components - for (Iterator iter = dirtyRegions.keySet().iterator(); iter.hasNext(); ) { - JComponent comp = (JComponent) iter.next(); - Rectangle rect = (Rectangle) dirtyRegions.get(comp); + for (Iterator iter = dirtyRegions.keySet().iterator(); iter.hasNext(); ) { + JComponent comp = iter.next(); + Rectangle rect = dirtyRegions.get(comp); RepaintManager rm = RepaintManager.currentManager(comp); rm.addDirtyRegion(comp, rect.x, rect.y, rect.width, rect.height); } // Draw all dirty regions - for (Iterator iter = repaintManagers.keySet().iterator(); iter.hasNext(); ) { - ((RepaintManager) iter.next()).paintDirtyRegions(); + for (Iterator iter = repaintManagers.keySet().iterator(); iter.hasNext(); ) { + iter.next().paintDirtyRegions(); } dirtyRegions.clear(); repaintManagers.clear(); diff --git a/src/jogl/classes/com/jogamp/opengl/util/AnimatorBase.java b/src/jogl/classes/com/jogamp/opengl/util/AnimatorBase.java index a6ba74665..e84a9bf78 100644 --- a/src/jogl/classes/com/jogamp/opengl/util/AnimatorBase.java +++ b/src/jogl/classes/com/jogamp/opengl/util/AnimatorBase.java @@ -28,6 +28,7 @@ package com.jogamp.opengl.util; +import com.jogamp.common.util.locks.LockFactory; import com.jogamp.common.util.locks.RecursiveLock; import jogamp.opengl.Debug; import jogamp.opengl.FPSCounterImpl; @@ -35,7 +36,6 @@ import jogamp.opengl.FPSCounterImpl; import java.io.PrintStream; import java.util.ArrayList; -import javax.media.opengl.FPSCounter; import javax.media.opengl.GLAnimatorControl; import javax.media.opengl.GLAutoDrawable; import javax.media.opengl.GLProfile; @@ -55,11 +55,11 @@ public abstract class AnimatorBase implements GLAnimatorControl { private static int animatorCount = 0; public interface AnimatorImpl { - void display(ArrayList drawables, boolean ignoreExceptions, boolean printExceptions); + void display(ArrayList drawables, boolean ignoreExceptions, boolean printExceptions); boolean skipWaitForCompletion(Thread thread); } - protected ArrayList/**/ drawables = new ArrayList(); + protected ArrayList drawables = new ArrayList(); protected boolean drawablesEmpty; protected AnimatorImpl impl; protected String baseName; @@ -67,7 +67,7 @@ public abstract class AnimatorBase implements GLAnimatorControl { protected boolean ignoreExceptions; protected boolean printExceptions; protected FPSCounterImpl fpsCounter = new FPSCounterImpl(); - protected RecursiveLock stateSync = new RecursiveLock(); + protected RecursiveLock stateSync = LockFactory.createRecursiveLock(); /** Creates a new, empty Animator. */ public AnimatorBase() { diff --git a/src/jogl/classes/com/jogamp/opengl/util/DefaultAnimatorImpl.java b/src/jogl/classes/com/jogamp/opengl/util/DefaultAnimatorImpl.java index d3f9cdeeb..bad268f70 100644 --- a/src/jogl/classes/com/jogamp/opengl/util/DefaultAnimatorImpl.java +++ b/src/jogl/classes/com/jogamp/opengl/util/DefaultAnimatorImpl.java @@ -41,11 +41,11 @@ import javax.media.opengl.GLAutoDrawable; up this behavior if desired. */ class DefaultAnimatorImpl implements AnimatorBase.AnimatorImpl { - public void display(ArrayList drawables, + public void display(ArrayList drawables, boolean ignoreExceptions, boolean printExceptions) { for (int i=0; i */ +@SuppressWarnings("serial") public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosingProtocol { private static final boolean DEBUG; @@ -498,7 +500,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing } } - RecursiveLock drawableSync = new RecursiveLock(); + RecursiveLock drawableSync = LockFactory.createRecursiveLock(); /** Overridden to track when this component is added to a container. Subclasses which override this method must call @@ -507,6 +509,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing Overrides:
addNotify in class java.awt.Component
*/ + @SuppressWarnings("deprecation") @Override public void addNotify() { if(DEBUG) { @@ -593,6 +596,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing about this.

Overrides:
removeNotify in class java.awt.Component
*/ + @SuppressWarnings("deprecation") @Override public void removeNotify() { if(DEBUG) { @@ -628,6 +632,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing Overrides:
reshape in class java.awt.Component
*/ + @SuppressWarnings("deprecation") @Override public void reshape(int x, int y, int width, int height) { super.reshape(x, y, width, height); @@ -935,10 +940,10 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing private void disableBackgroundErase() { if (!disableBackgroundEraseInitialized) { try { - AccessController.doPrivileged(new PrivilegedAction() { + AccessController.doPrivileged(new PrivilegedAction() { public Object run() { try { - Class clazz = getToolkit().getClass(); + Class clazz = getToolkit().getClass(); while (clazz != null && disableBackgroundEraseMethod == null) { try { disableBackgroundEraseMethod = @@ -1006,7 +1011,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing chooser, aScreen); } else { try { - final ArrayList bucket = new ArrayList(1); + final ArrayList bucket = new ArrayList(1); EventQueue.invokeAndWait(new Runnable() { public void run() { AWTGraphicsConfiguration c = (AWTGraphicsConfiguration) @@ -1016,7 +1021,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing bucket.add(c); } }); - config = ( bucket.size() > 0 ) ? (AWTGraphicsConfiguration)bucket.get(0) : null ; + config = ( bucket.size() > 0 ) ? bucket.get(0) : null ; } catch (InvocationTargetException e) { throw new GLException(e.getTargetException()); } catch (InterruptedException e) { @@ -1042,7 +1047,7 @@ public class GLCanvas extends Canvas implements AWTGLAutoDrawable, WindowClosing GLProfile.initSingleton(false); GLDrawableFactory factory = GLDrawableFactory.getDesktopFactory(); - List/**/ availCaps = factory.getAvailableCapabilities(null); + List availCaps = factory.getAvailableCapabilities(null); for(int i=0; i