Bugzilla 46883: fixed a hotspot in the AFP renderer. Reduced time spent in the method by introducing a member variable to hold the data-length.

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@807010 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 10 insertions, 9 deletions

diff --git a/src/java/org/apache/fop/afp/goca/AbstractGraphicsDrawingOrderContainer.java b/src/java/org/apache/fop/afp/goca/AbstractGraphicsDrawingOrderContainer.java
index 34398b094..e700d76b2 100644
--- a/src/java/org/apache/fop/afp/goca/AbstractGraphicsDrawingOrderContainer.java
+++ b/src/java/org/apache/fop/afp/goca/AbstractGraphicsDrawingOrderContainer.java
@@ -46,6 +46,8 @@ implements StructuredData, Completable, Startable {
     /** object has started */
     private boolean started = false;
 
+    private int dataLength = 0;
+
     /**
      * Default constructor
      */
@@ -78,6 +80,7 @@ implements StructuredData, Completable, Startable {
      */
     public void addObject(StructuredData object) {
         objects.add(object);
+        dataLength += object.getDataLength();
     }
 
     /**
@@ -88,6 +91,7 @@ implements StructuredData, Completable, Startable {
     public void addAll(AbstractGraphicsDrawingOrderContainer graphicsContainer) {
         Collection/*<StructuredDataObject>*/ objects = graphicsContainer.getObjects();
         objects.addAll(objects);
+        dataLength += graphicsContainer.getDataLength();
     }
 
     /**
@@ -107,9 +111,11 @@ implements StructuredData, Completable, Startable {
     public StructuredData removeLast() {
         int lastIndex = objects.size() - 1;
         StructuredData object = null;
-        if (lastIndex > -1) {
-            object = (StructuredData)objects.get(lastIndex);
-            objects.remove(lastIndex);
+        if (lastIndex >= 0) {
+            object = (StructuredData)objects.remove(lastIndex);
+        }
+        if (object != null) {
+            dataLength -= object.getDataLength();
         }
         return object;
     }
@@ -121,12 +127,7 @@ implements StructuredData, Completable, Startable {
      * all enclosed objects (and their containers)
      */
     public int getDataLength() {
-        int dataLen = 0;
-        Iterator it = objects.iterator();
-        while (it.hasNext()) {
-            dataLen += ((StructuredData)it.next()).getDataLength();
-        }
-        return dataLen;
+        return this.dataLength;
     }
 
     /** {@inheritDoc} */