From dce77f00239074d33bb7dca4009e45ba87141093 Mon Sep 17 00:00:00 2001 From: Rami Santina Date: Sun, 5 Jun 2011 22:05:38 +0300 Subject: Graph: updated inclass documentations --- .../classes/com/jogamp/graph/curve/Region.java | 86 ++++++++++++++-------- .../com/jogamp/graph/curve/opengl/GLRegion.java | 6 +- .../com/jogamp/graph/curve/opengl/Renderer.java | 2 +- 3 files changed, 58 insertions(+), 36 deletions(-) (limited to 'src/jogl/classes/com/jogamp/graph') diff --git a/src/jogl/classes/com/jogamp/graph/curve/Region.java b/src/jogl/classes/com/jogamp/graph/curve/Region.java index 168d8dd70..af15f9dc4 100644 --- a/src/jogl/classes/com/jogamp/graph/curve/Region.java +++ b/src/jogl/classes/com/jogamp/graph/curve/Region.java @@ -35,8 +35,18 @@ import com.jogamp.graph.geom.AABBox; import com.jogamp.graph.geom.Triangle; import com.jogamp.graph.geom.Vertex; +/** Abstract Outline shape GL representation + * define the method an OutlineShape(s) is + * binded rendered. + * + * @see GLRegion + */ public abstract class Region { + + /** Debug flag for region impl (graph.curve) + */ public static final boolean DEBUG = Debug.debug("graph.curve"); + public static final boolean DEBUG_INSTANCE = false; /** View based Anti-Aliasing, A Two pass region rendering, slower @@ -46,36 +56,57 @@ public abstract class Region { public static final int VBAA_RENDERING_BIT = 1 << 0; /** Use non uniform weights [0.0 .. 1.9] for curve region rendering. - * Otherwise the default weight 1.0 for Bezier curve region rendering is being applied. */ + * Otherwise the default weight 1.0 for uniform curve region rendering is being applied. + */ public static final int VARIABLE_CURVE_WEIGHT_BIT = 1 << 1; - + public static final int TWO_PASS_DEFAULT_TEXTURE_UNIT = 0; - + private final int renderModes; private boolean dirty = true; protected int numVertices = 0; - protected boolean flipped = false; protected final AABBox box = new AABBox(); protected ArrayList triangles = new ArrayList(); protected ArrayList vertices = new ArrayList(); - + public static boolean isVBAA(int renderModes) { return 0 != ( renderModes & Region.VBAA_RENDERING_BIT ); } - - public static boolean usesVariableCurveWeight(int renderModes) { + + /** Check if render mode capable of non uniform weights + * @param renderModes bit-field of modes, e.g. {@link Region#VARIABLE_CURVE_WEIGHT_BIT}, + * {@link Region#VBAA_RENDERING_BIT} + * @return true of capable of non uniform weights + */ + public static boolean isNonUniformWeight(int renderModes) { return 0 != ( renderModes & Region.VARIABLE_CURVE_WEIGHT_BIT ); } - + protected Region(int regionRenderModes) { this.renderModes = regionRenderModes; } - - public final int getRenderModes() { return renderModes; } - public boolean isVBAA() { return Region.isVBAA(renderModes); } - public boolean usesVariableCurveWeight() { return Region.usesVariableCurveWeight(renderModes); } - + /** Get current Models + * @return bit-field of render modes + */ + public final int getRenderModes() { + return renderModes; + } + + /** Check if current Region is using VBAA + * @return true if capable of two pass rendering - VBAA + */ + public boolean isVBAA() { + return Region.isVBAA(renderModes); + } + + /** Check if current instance uses non uniform weights + * @return true if capable of nonuniform weights + */ + public boolean isNonUniformWeight() { + return Region.isNonUniformWeight(renderModes); + } + /** Get the current number of vertices associated * with this region. This number is not necessary equal to * the OGL bound number of vertices. @@ -84,7 +115,7 @@ public abstract class Region { public final int getNumVertices(){ return numVertices; } - + /** Adds a {@link Triangle} object to the Region * This triangle will be bound to OGL objects * on the next call to {@code update} @@ -96,7 +127,7 @@ public abstract class Region { triangles.add(tri); setDirty(true); } - + /** Adds a list of {@link Triangle} objects to the Region * These triangles are to be binded to OGL objects * on the next call to {@code update} @@ -108,7 +139,7 @@ public abstract class Region { triangles.addAll(tris); setDirty(true); } - + /** Adds a {@link Vertex} object to the Region * This vertex will be bound to OGL objects * on the next call to {@code update} @@ -121,7 +152,7 @@ public abstract class Region { numVertices++; setDirty(true); } - + /** Adds a list of {@link Vertex} objects to the Region * These vertices are to be binded to OGL objects * on the next call to {@code update} @@ -134,24 +165,15 @@ public abstract class Region { numVertices = vertices.size(); setDirty(true); } - + + /** + * @return the AxisAligned bounding box of + * current region + */ public final AABBox getBounds(){ return box; } - /** Set if the y coordinate of the region should be flipped - * {@code y=-y} used mainly for fonts since they use opposite vertex - * as origion - * @param flipped flag if the coordinate is flipped defaults to false. - */ - public void setFlipped(boolean flipped) { - this.flipped = flipped; - } - - public final boolean isFlipped() { - return flipped; - } - /** Check if this region is dirty. A region is marked dirty * when new Vertices, Triangles, and or Lines are added after a * call to update() @@ -162,7 +184,7 @@ public abstract class Region { public final boolean isDirty() { return dirty; } - + protected final void setDirty(boolean v) { dirty = v; } diff --git a/src/jogl/classes/com/jogamp/graph/curve/opengl/GLRegion.java b/src/jogl/classes/com/jogamp/graph/curve/opengl/GLRegion.java index c6165b6b3..749c7ef65 100755 --- a/src/jogl/classes/com/jogamp/graph/curve/opengl/GLRegion.java +++ b/src/jogl/classes/com/jogamp/graph/curve/opengl/GLRegion.java @@ -39,15 +39,15 @@ import com.jogamp.graph.geom.Triangle; import com.jogamp.graph.geom.Vertex; import jogamp.graph.curve.opengl.RegionFactory; -/** A Region is the OGL binding of one or more OutlineShapes +/** A GLRegion is the OGL binding of one or more OutlineShapes * Defined by its vertices and generated triangles. The Region * defines the final shape of the OutlineShape(s), which shall produced a shaded * region on the screen. * - * Implementations of the Region shall take care of the OGL + * Implementations of the GLRegion shall take care of the OGL * binding of the depending on its context, profile. * - * @see RegionFactory, OutlineShape + * @see Region, RegionFactory, OutlineShape */ public abstract class GLRegion extends Region { diff --git a/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java b/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java index e45cacbf6..f1d31c345 100644 --- a/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java +++ b/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java @@ -85,7 +85,7 @@ public abstract class Renderer { return renderModes; } - public boolean usesVariableCurveWeight() { return Region.usesVariableCurveWeight(renderModes); } + public boolean usesVariableCurveWeight() { return Region.isNonUniformWeight(renderModes); } /** * @return true if Region's renderModes contains all bits as this Renderer's renderModes -- cgit v1.2.3 From d23d1dd01cc7e3bb228d65feb8f781c4ff0e5e11 Mon Sep 17 00:00:00 2001 From: Rami Santina Date: Sun, 5 Jun 2011 23:12:36 +0300 Subject: Graph: Remove unused imports --- src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java | 1 - src/jogl/classes/com/jogamp/graph/font/FontFactory.java | 1 - src/jogl/classes/jogamp/graph/curve/opengl/VBORegionSPES2.java | 1 - src/jogl/classes/jogamp/graph/font/typecast/TypecastRenderer.java | 1 - .../com/jogamp/opengl/test/junit/graph/demos/GPURegionGLListener01.java | 1 - 5 files changed, 5 deletions(-) (limited to 'src/jogl/classes/com/jogamp/graph') diff --git a/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java b/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java index f1d31c345..e77fc14d4 100644 --- a/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java +++ b/src/jogl/classes/com/jogamp/graph/curve/opengl/Renderer.java @@ -37,7 +37,6 @@ import com.jogamp.opengl.util.glsl.ShaderState; import com.jogamp.opengl.util.PMVMatrix; import com.jogamp.graph.curve.Region; -import com.jogamp.graph.geom.Vertex; public abstract class Renderer { protected static final boolean DEBUG = Region.DEBUG; diff --git a/src/jogl/classes/com/jogamp/graph/font/FontFactory.java b/src/jogl/classes/com/jogamp/graph/font/FontFactory.java index 9bf4b1f73..1e83668de 100644 --- a/src/jogl/classes/com/jogamp/graph/font/FontFactory.java +++ b/src/jogl/classes/com/jogamp/graph/font/FontFactory.java @@ -29,7 +29,6 @@ package com.jogamp.graph.font; import java.io.File; import java.io.IOException; -import java.lang.reflect.InvocationTargetException; import java.net.URL; import java.security.AccessController; import java.security.PrivilegedAction; diff --git a/src/jogl/classes/jogamp/graph/curve/opengl/VBORegionSPES2.java b/src/jogl/classes/jogamp/graph/curve/opengl/VBORegionSPES2.java index bad436a80..dc4e3a6e0 100644 --- a/src/jogl/classes/jogamp/graph/curve/opengl/VBORegionSPES2.java +++ b/src/jogl/classes/jogamp/graph/curve/opengl/VBORegionSPES2.java @@ -34,7 +34,6 @@ import jogamp.graph.curve.opengl.shader.AttributeNames; import com.jogamp.graph.curve.opengl.GLRegion; import com.jogamp.graph.curve.opengl.RenderState; -import com.jogamp.graph.geom.AABBox; import com.jogamp.graph.geom.Vertex; import com.jogamp.graph.geom.Triangle; import com.jogamp.opengl.util.GLArrayDataServer; diff --git a/src/jogl/classes/jogamp/graph/font/typecast/TypecastRenderer.java b/src/jogl/classes/jogamp/graph/font/typecast/TypecastRenderer.java index 50b510d23..86d4d2a3d 100644 --- a/src/jogl/classes/jogamp/graph/font/typecast/TypecastRenderer.java +++ b/src/jogl/classes/jogamp/graph/font/typecast/TypecastRenderer.java @@ -29,7 +29,6 @@ package jogamp.graph.font.typecast; import java.util.ArrayList; -import jogamp.graph.curve.text.GlyphShape; import jogamp.graph.font.typecast.ot.OTGlyph; import jogamp.graph.font.typecast.ot.Point; import jogamp.graph.geom.plane.AffineTransform; diff --git a/src/test/com/jogamp/opengl/test/junit/graph/demos/GPURegionGLListener01.java b/src/test/com/jogamp/opengl/test/junit/graph/demos/GPURegionGLListener01.java index e5e3c56e5..b8bb9ac14 100644 --- a/src/test/com/jogamp/opengl/test/junit/graph/demos/GPURegionGLListener01.java +++ b/src/test/com/jogamp/opengl/test/junit/graph/demos/GPURegionGLListener01.java @@ -32,7 +32,6 @@ import javax.media.opengl.GL; import javax.media.opengl.GL2ES2; import javax.media.opengl.GLAutoDrawable; import com.jogamp.graph.curve.OutlineShape; -import com.jogamp.graph.curve.Region; import com.jogamp.graph.curve.opengl.GLRegion; import com.jogamp.graph.curve.opengl.RegionRenderer; import com.jogamp.graph.curve.opengl.RenderState; -- cgit v1.2.3