From 67973c43fb1f6873b19b327b3c665452e5131960 Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Fri, 5 May 2023 03:13:53 +0200 Subject: Graph GLRegion: DEBUG_BUFFER (growth) 'jogl.debug.graph.curve.Buffer' --- .../com/jogamp/graph/curve/opengl/GLRegion.java | 54 ++++++++++++++-------- 1 file changed, 35 insertions(+), 19 deletions(-) (limited to 'src/jogl/classes') 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 668223f25..49daa344b 100644 --- a/src/jogl/classes/com/jogamp/graph/curve/opengl/GLRegion.java +++ b/src/jogl/classes/com/jogamp/graph/curve/opengl/GLRegion.java @@ -42,6 +42,7 @@ import jogamp.graph.curve.opengl.VBORegion2PMSAAES2; import jogamp.graph.curve.opengl.VBORegion2PVBAAES2; import jogamp.graph.curve.opengl.VBORegionSPES2; import jogamp.graph.curve.opengl.shader.AttributeNames; +import jogamp.opengl.Debug; import com.jogamp.opengl.util.PMVMatrix; import com.jogamp.opengl.util.glsl.ShaderProgram; @@ -94,6 +95,8 @@ public abstract class GLRegion extends Region { // private static final float growthFactor = 1.2f; // avg +5% size but 15% more overhead (34% total) protected static final float growthFactor = GLArrayDataClient.DEFAULT_GROWTH_FACTOR; // avg +20% size, but 15% less CPU overhead compared to 1.2 (19% total) + private static final boolean DEBUG_BUFFER = Debug.debug("graph.curve.Buffer"); + /** * Create a GLRegion using the passed render mode * @@ -228,31 +231,44 @@ public abstract class GLRegion extends Region { growCount = 0; } - private static final boolean DEBUG_BUFFER = false; - @Override public final boolean growBuffer(final int verticesCount, final int indicesCount) { boolean grown = false; - if( curIndicesCap < indicesBuffer.elemPosition() + indicesCount ) { - System.err.printf("XXX Buffer grow - Indices: %d < ( %d = %d + %d ); Status: %s%n", - curIndicesCap, indicesBuffer.elemPosition() + indicesCount, indicesBuffer.elemPosition(), indicesCount, indicesBuffer.elemStatsToString()); - indicesBuffer.growIfNeeded(indicesCount * indicesBuffer.getCompsPerElem()); - if( DEBUG_BUFFER ) { - System.err.println("grew.indices 0x"+Integer.toHexString(hashCode())+": "+curIndicesCap+" -> "+indicesBuffer.getElemCapacity()+", "+indicesBuffer.elemStatsToString()); + if( !DEBUG_BUFFER ) { + if( curIndicesCap < indicesBuffer.elemPosition() + indicesCount ) { + indicesBuffer.growIfNeeded(indicesCount * indicesBuffer.getCompsPerElem()); + curIndicesCap = indicesBuffer.getElemCapacity(); + grown = true; + } + if( curVerticesCap < vpc_ileave.elemPosition() + verticesCount ) { + vpc_ileave.growIfNeeded(verticesCount * vpc_ileave.getCompsPerElem()); + curVerticesCap = vpc_ileave.getElemCapacity(); + grown = true; + } + } else { + if( curIndicesCap < indicesBuffer.elemPosition() + indicesCount ) { + System.err.printf("GLRegion: Buffer grow - Indices: %d < ( %d = %d + %d ); Status: %s%n", + curIndicesCap, indicesBuffer.elemPosition() + indicesCount, indicesBuffer.elemPosition(), indicesCount, indicesBuffer.elemStatsToString()); + + indicesBuffer.growIfNeeded(indicesCount * indicesBuffer.getCompsPerElem()); + + System.err.println("GLRegion: Grew Indices 0x"+Integer.toHexString(hashCode())+": "+curIndicesCap+" -> "+indicesBuffer.getElemCapacity()+", "+indicesBuffer.elemStatsToString()); Thread.dumpStack(); + + curIndicesCap = indicesBuffer.getElemCapacity(); + grown = true; } - curIndicesCap = indicesBuffer.getElemCapacity(); - grown = true; - } - if( curVerticesCap < vpc_ileave.elemPosition() + verticesCount ) { - System.err.printf("XXX Buffer grow - Verices: %d < ( %d = %d + %d ); Status: %s%n", - curVerticesCap, gca_VerticesAttr.elemPosition() + verticesCount, gca_VerticesAttr.elemPosition(), verticesCount, gca_VerticesAttr.elemStatsToString()); - vpc_ileave.growIfNeeded(verticesCount * vpc_ileave.getCompsPerElem()); - if( DEBUG_BUFFER ) { - System.err.println("grew.vertices 0x"+Integer.toHexString(hashCode())+": "+curVerticesCap+" -> "+gca_VerticesAttr.getElemCapacity()+", "+gca_VerticesAttr.elemStatsToString()); + if( curVerticesCap < vpc_ileave.elemPosition() + verticesCount ) { + System.err.printf("GLRegion: Buffer grow - Vertices: %d < ( %d = %d + %d ); Status: %s%n", + curVerticesCap, gca_VerticesAttr.elemPosition() + verticesCount, gca_VerticesAttr.elemPosition(), verticesCount, gca_VerticesAttr.elemStatsToString()); + + vpc_ileave.growIfNeeded(verticesCount * vpc_ileave.getCompsPerElem()); + + System.err.println("GLRegion: Grew Vertices 0x"+Integer.toHexString(hashCode())+": "+curVerticesCap+" -> "+gca_VerticesAttr.getElemCapacity()+", "+gca_VerticesAttr.elemStatsToString()); + + curVerticesCap = vpc_ileave.getElemCapacity(); + grown = true; } - curVerticesCap = vpc_ileave.getElemCapacity(); - grown = true; } if( grown ) { ++growCount; -- cgit v1.2.3