Bug 802: Graph TextRenderer Performance Part-1 (incomplete, rendering artifacts)

Strategy Change:
  - Font.Glyph itself holds it's OutlineShape
    with it's default scaling.
    Triangulation is done only once per glyph!

  - A CharSequence produces a Region
    by translating and scaling each Glyphs's OutlineShape.
    This removes the need for re-triangulate - see above.
    See: TextRendererUtil

  - The indices of re-added Triangles are
    offset to the new vertices (FIXME, seems not be be accurate yet).

  - OutlineShape's vertices and triangles are reused if 'clean'.

  - Simplified code

  - Reduced copies

API Changes:
  - OutlineShape, Region, ...: See above

  - Removed TextRenderer, GlyphShape and GlyphString: Redundant

  - Added TextRendererUtil to produce the Region from CharSequence

Result:
  - Over 600 fps while changing text for each frame.
    Previously only ~60fps max.

TODO:
  - Region shall not hold the triangles itself,
    but the indices instead.
    This will remove the need to swizzle w/ vertices in the Region Renderer impl
    and easies reusage of OutlineShapes.

1 files changed, 58 insertions, 9 deletions

diff --git a/src/jogl/classes/com/jogamp/graph/geom/Triangle.java b/src/jogl/classes/com/jogamp/graph/geom/Triangle.java
index a01cd834f..6de99c48b 100644
--- a/src/jogl/classes/com/jogamp/graph/geom/Triangle.java
+++ b/src/jogl/classes/com/jogamp/graph/geom/Triangle.java
@@ -27,14 +27,52 @@
  */
 package com.jogamp.graph.geom;
 
+import com.jogamp.graph.curve.Region;
+
+import jogamp.graph.geom.plane.AffineTransform;
+
 public class Triangle {
-    private int id = Integer.MAX_VALUE;
-    final private Vertex[] vertices;
-    private boolean[] boundaryEdges = new boolean[3];
+    private final Vertex[] vertices = new Vertex[3];
+    private final boolean[] boundaryEdges = new boolean[3];
     private boolean[] boundaryVertices = null;
+    private int id;
+
+    public Triangle(Vertex v1, Vertex v2, Vertex v3) {
+        id = Integer.MAX_VALUE;
+        vertices[0] = v1;
+        vertices[1] = v2;
+        vertices[2] = v3;
+    }
+
+    public Triangle(Triangle src) {
+        id = src.id;
+        vertices[0] = src.vertices[0].clone();
+        vertices[1] = src.vertices[1].clone();
+        vertices[2] = src.vertices[2].clone();
+        System.arraycopy(src.boundaryEdges, 0, boundaryEdges, 0, 3);
+        boundaryVertices = src.boundaryVertices;
+    }
+
+    private Triangle(final int id, final boolean[] boundaryEdges, final boolean[] boundaryVertices){
+        this.id = id;
+        System.arraycopy(boundaryEdges, 0, this.boundaryEdges, 0, 3);
+        this.boundaryVertices = boundaryVertices;
+        /**
+        if( null != boundaryVertices ) {
+            this.boundaryVertices = new boolean[3];
+            System.arraycopy(boundaryVertices, 0, this.boundaryVertices, 0, 3);
+        } */
+    }
 
-    public Triangle(Vertex ... v123){
-        vertices = v123;
+    /**
+     * Returns a transformed a clone of this instance using the given AffineTransform.
+     */
+    public Triangle transform(AffineTransform t) {
+        final Triangle tri = new Triangle(id, boundaryEdges, boundaryVertices);
+        tri.vertices[0] = t.transform(vertices[0], null);
+        tri.vertices[1] = t.transform(vertices[1], null);
+        tri.vertices[2] = t.transform(vertices[2], null);
+        return tri;
     }
 
     public int getId() {
@@ -45,6 +83,21 @@ public class Triangle {
         this.id = id;
     }
 
+    public void addVertexIndicesOffset(int offset) {
+        if( 0 < offset ) {
+            final int i0 = vertices[0].getId();
+            if( Integer.MAX_VALUE-offset > i0 ) { // Integer.MAX_VALUE != i0 // FIXME: renderer uses SHORT!
+                if(Region.DEBUG_INSTANCE) {
+                    System.err.println("Triangle.addVertexIndicesOffset: "+i0+" + "+offset+" -> "+(i0+offset));
+                }
+                vertices[0].setId(i0+offset);
+                vertices[1].setId(vertices[1].getId()+offset);
+                vertices[2].setId(vertices[2].getId()+offset);
+            }
+        }
+    }
+
+    /** Returns array of 3 vertices, denominating the triangle. */
     public Vertex[] getVertices() {
         return vertices;
     }
@@ -57,10 +110,6 @@ public class Triangle {
         return boundaryVertices[0] || boundaryVertices[1] || boundaryVertices[2];
     }
 
-    public void setEdgesBoundary(boolean[] boundary) {
-        this.boundaryEdges = boundary;
-    }
-
     public boolean[] getEdgeBoundary() {
         return boundaryEdges;
     }