]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
PR:
authorGlen Mazza <gmazza@apache.org>
Wed, 4 Aug 2004 22:21:04 +0000 (22:21 +0000)
committerGlen Mazza <gmazza@apache.org>
Wed, 4 Aug 2004 22:21:04 +0000 (22:21 +0000)
Obtained from:
Submitted by:
Reviewed by:
Start of conversion of AddLMVisitor to FObj.AddLayoutManager():  removed
acceptVisitor() methods from FO's that don't need LM's
(e.g., those FO's not yet implemented or those that don't return areas anyway).

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

18 files changed:
src/java/org/apache/fop/fo/FObj.java
src/java/org/apache/fop/fo/ToBeImplementedElement.java
src/java/org/apache/fop/fo/Unknown.java
src/java/org/apache/fop/fo/UnknownXMLObj.java
src/java/org/apache/fop/fo/XMLElement.java
src/java/org/apache/fop/fo/XMLObj.java
src/java/org/apache/fop/fo/extensions/svg/SVGElement.java
src/java/org/apache/fop/fo/extensions/svg/SVGObj.java
src/java/org/apache/fop/fo/flow/Float.java
src/java/org/apache/fop/fo/flow/InitialPropertySet.java
src/java/org/apache/fop/fo/flow/MultiCase.java
src/java/org/apache/fop/fo/flow/MultiProperties.java
src/java/org/apache/fop/fo/flow/MultiPropertySet.java
src/java/org/apache/fop/fo/flow/MultiSwitch.java
src/java/org/apache/fop/fo/flow/MultiToggle.java
src/java/org/apache/fop/fo/flow/TableAndCaption.java
src/java/org/apache/fop/fo/flow/TableCaption.java
src/java/org/apache/fop/layoutmgr/AddLMVisitor.java

index 3d8c69b1b071d4f189298aa2e034c0155970a7d3..3d785d399b680cee1d42f5cf13e28655bac67c9b 100644 (file)
@@ -21,6 +21,7 @@ package org.apache.fop.fo;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Iterator;
+import java.util.List;
 import java.util.ListIterator;
 import java.util.Map;
 import java.util.Set;
@@ -433,6 +434,14 @@ public class FObj extends FONode implements Constants {
         return markers;
     }
 
+    /**
+     * Return a LayoutManager responsible for laying out this FObj's content.
+     * Must override in subclasses if their content can be laid out.
+     * @param list the list to which the layout manager(s) should be added
+     */
+    public void addLayoutManager(List list) {
+    }
+
     /**
      * This is a hook for the AddLMVisitor class to be able to access
      * this object.
@@ -442,7 +451,6 @@ public class FObj extends FONode implements Constants {
         aLMV.serveFObj(this);
     }
     
-
     /*
      * Return a string representation of the fo element.
      * Deactivated in order to see precise ID of each fo element created
index 7c77741f47c5690417069341494ff5bef8f07da4..d445fe1e426e5372bcad64a31905a6f34524d56d 100644 (file)
@@ -18,8 +18,6 @@
 
 package org.apache.fop.fo;
 
-import org.apache.fop.layoutmgr.AddLMVisitor;
-
 /**
  * This class is a placeholder for elements that have not been implemented.
  */
@@ -31,14 +29,4 @@ public class ToBeImplementedElement extends FObj {
     protected ToBeImplementedElement(FONode parent) {
         super(parent);
     }
-
-    /**
-     * This is a hook for the AddLMVisitor class to be able to access
-     * this object.
-     * @param aLMV the AddLMVisitor object that can access this object.
-     */
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveToBeImplementedElement(this);
-    }
-
 }
index 2a8edee8cb801de4c5ccc037e3674b9b2e7891ed..e3d063a8683a5ae9e310103101adb2e8e8351bef 100644 (file)
@@ -18,8 +18,6 @@
 
 package org.apache.fop.fo;
 
-import org.apache.fop.layoutmgr.AddLMVisitor;
-
 /**
  * Class for handling an unknown element, for example one from an unsupported
  * namespace.
@@ -53,14 +51,4 @@ public class Unknown extends FONode {
     private void setup() {
         getLogger().debug("Layout Unknown element");
     }
-
-    /**
-     * This is a hook for the AddLMVisitor class to be able to access
-     * this object.
-     * @param aLMV the AddLMVisitor object that can access this object.
-     */
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveUnknown(this);
-    }
-
 }
index a64759b87ee2937bd6725c8e3a6d65cb8cb6e1aa..e768973c3ce5cb2145df6029e3f61e0e40064843 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo;
 
 import org.xml.sax.Locator;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 
 /**
  * Class for handling generic XML from a namespace not recognized by FOP
@@ -89,15 +88,5 @@ public class UnknownXMLObj extends XMLObj {
         }
         super.addCharacters(data, start, length, locator);
     }
-
-    /**
-     * This is a hook for the AddLMVisitor class to be able to access
-     * this object.
-     * @param aLMV the AddLMVisitor object that can access this object.
-     */
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveUnknownXMLObj(this);
-    }
-
 }
 
index accd4f45ae520bdbce3a21e8f778b5e03f100e79..4f9431c3e7f0b09b1222fd376afadad37b290245 100644 (file)
@@ -23,9 +23,6 @@ import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
-// FOP
-import org.apache.fop.layoutmgr.AddLMVisitor;
-
 /**
  * class representing svg:svg pseudo flow object.
  */
@@ -62,14 +59,4 @@ public class XMLElement extends XMLObj {
     public String getNameSpace() {
         return namespace;
     }
-
-    /**
-     * This is a hook for the AddLMVisitor class to be able to access
-     * this object.
-     * @param aLMV the AddLMVisitor object that can access this object.
-     */
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveXMLElement(this);
-    }
-
 }
index f48d2077fcd43fd035275aa9f40ac332f5684528..b87ae8dcd1aa8e67f6ad6b707add8cce08079ab4 100644 (file)
@@ -29,9 +29,6 @@ import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 import javax.xml.parsers.DocumentBuilderFactory;
 
-// FOP
-import org.apache.fop.layoutmgr.AddLMVisitor;
-
 /**
  * Abstract class modelling generic, non-XSL-FO XML objects. Such objects are
  * stored in a DOM.
@@ -209,15 +206,5 @@ public abstract class XMLObj extends FONode {
         org.w3c.dom.Text text = doc.createTextNode(str);
         element.appendChild(text);
     }
-
-    /**
-     * This is a hook for the AddLMVisitor class to be able to access
-     * this object.
-     * @param aLMV the AddLMVisitor object that can access this object.
-     */
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveXMLObj(this);
-    }
-
 }
 
index 1575bacc7b4b2f6e1a379ddcd7bd58561f1f352d..67923a7afa4f68eaa6a854058028236af3cfc346 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.extensions.svg;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 
 import org.apache.batik.dom.svg.SVGOMDocument;
 import org.apache.batik.dom.svg.SVGOMElement;
@@ -282,10 +281,5 @@ public class SVGElement extends SVGObj {
             return 100;
         }
     }
-
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveSVGElement(this);
-    }
-
 }
 
index bf932df6df4f04a28d023aa0d27229ff4a02fb10..0d300b9a4f921fc7d753de045531c78676ffec17 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.extensions.svg;
 
 import org.apache.fop.fo.FONode;
 import org.apache.fop.fo.XMLObj;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 
 /**
  * Class for SVG element objects.
@@ -43,10 +42,5 @@ public class SVGObj extends XMLObj {
     public String getNameSpace() {
         return "http://www.w3.org/2000/svg";
     }
-
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveSVGObj(this);
-    }
-
 }
 
index e92503262facd4f2ba41c1352fe3a644fcf8efbd..a428e86ca78828a13175d149033e97ca9f649d45 100644 (file)
@@ -21,7 +21,6 @@ package org.apache.fop.fo.flow;
 // FOP
 import org.apache.fop.fo.FONode;
 import org.apache.fop.fo.ToBeImplementedElement;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 
 /**
  * fo:float element.
@@ -42,10 +41,6 @@ public class Float extends ToBeImplementedElement {
 
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveFloat(this);
-    }
-
     public String getName() {
         return "fo:float";
     }
index 9bc17ff6bbdbc4b5c321a60fb216c3ebc4126c83..2972491ca551d216a7dc54460c2742de386254e3 100644 (file)
@@ -25,7 +25,6 @@ import org.xml.sax.SAXParseException;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 import org.apache.fop.fo.properties.CommonAural;
@@ -86,10 +85,6 @@ public class InitialPropertySet extends ToBeImplementedElement {
 
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveInitialPropertySet(this);
-    }
-
     public String getName() {
         return "fo:initial-property-set";
     }
index 52cdb0daa1ed9185d6db251f7b25b5529c0efb06..efac25394116b0185f591c465db9cb9ed741f010 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.flow;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 
@@ -49,10 +48,6 @@ public class MultiCase extends ToBeImplementedElement {
 
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveMultiCase(this);
-    }
-
     public String getName() {
         return "fo:multi-case";
     }
index 86df3e4cf71ef6d9535cd9eb5107771b76f65948..a2a2c04e503960c73f6cba8f1f0f85e53826b809 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.flow;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 
@@ -46,10 +45,6 @@ public class MultiProperties extends ToBeImplementedElement {
 
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveMultiProperties(this);
-    }
-
     public String getName() {
         return "fo:multi-properties";
     }
index cc971388d9733a1fb6253d3f71adda1bf80f869b..05edd6e019c9f49ccfb5e18b21c8457bfbcedf11 100644 (file)
@@ -25,7 +25,6 @@ import org.xml.sax.SAXParseException;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 
 /**
@@ -55,10 +54,6 @@ public class MultiPropertySet extends ToBeImplementedElement {
         // this.propertyList.get("active-state");
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveMultiPropertySet(this);
-    }
-
     public String getName() {
         return "fo:multi-property-set";
     }
index 0ce8d34ba8d0ac51e2885b51384c88158b29160b..61420201ac5ab04f38128a7b938f468e704f9f5e 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.flow;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 
@@ -47,10 +46,6 @@ public class MultiSwitch extends ToBeImplementedElement {
 
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveMultiSwitch(this);
-    }
-
     public String getName() {
         return "fo:multi-switch";
     }
index f0abecde62d4a201c076c6907a106d16b1932060..06e129c268dfbadf27c6c1245bf72d023eb9a5cd 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.flow;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 
@@ -47,10 +46,6 @@ public class MultiToggle extends ToBeImplementedElement {
 
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveMultiToggle(this);
-    }
-
     public String getName() {
         return "fo:multi-toggle";
     }
index a903d358d1429936058babbac3724220d8094b54..b87b23c8c36ebfea55aac3be47448992408ea28c 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.flow;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 import org.apache.fop.fo.properties.CommonAural;
@@ -82,13 +81,8 @@ public class TableAndCaption extends ToBeImplementedElement {
         return true;
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveTableAndCaption(this);
-    }
-
     public String getName() {
         return "fo:table-and-caption";
     }
-
 }
 
index 2b29caef575655f8d77e5aaaa0fe7f19ae522824..ec8cacfe4245ead69398d5c1889f33f8da4ebf79 100644 (file)
@@ -20,7 +20,6 @@ package org.apache.fop.fo.flow;
 
 // FOP
 import org.apache.fop.fo.FONode;
-import org.apache.fop.layoutmgr.AddLMVisitor;
 import org.apache.fop.fo.ToBeImplementedElement;
 import org.apache.fop.fo.properties.CommonAccessibility;
 import org.apache.fop.fo.properties.CommonAural;
@@ -75,10 +74,6 @@ public class TableCaption extends ToBeImplementedElement {
         return true;
     }
 
-    public void acceptVisitor(AddLMVisitor aLMV) {
-        aLMV.serveTableCaption(this);
-    }
-
     public String getName() {
         return "fo:table-caption";
     }
index 3b30e81c64ffb8fd9fd0dcb559cd8ea7b83806e1..372c62d491ecad62dc84934cc0f016702c0aad9c 100644 (file)
@@ -47,26 +47,19 @@ import org.apache.fop.fo.FObj;
 import org.apache.fop.fo.FObjMixed;
 import org.apache.fop.fo.TextInfo;
 import org.apache.fop.fo.ToBeImplementedElement;
-import org.apache.fop.fo.Unknown;
-import org.apache.fop.fo.UnknownXMLObj;
-import org.apache.fop.fo.XMLElement;
 import org.apache.fop.fo.XMLObj;
 import org.apache.fop.fo.extensions.Bookmarks;
 import org.apache.fop.fo.extensions.ExtensionObj;
 import org.apache.fop.fo.extensions.Label;
 import org.apache.fop.fo.extensions.Outline;
-import org.apache.fop.fo.extensions.svg.SVGElement;
-import org.apache.fop.fo.extensions.svg.SVGObj;
 import org.apache.fop.fo.flow.BasicLink;
 import org.apache.fop.fo.flow.BidiOverride;
 import org.apache.fop.fo.flow.Block;
 import org.apache.fop.fo.flow.BlockContainer;
 import org.apache.fop.fo.flow.Character;
 import org.apache.fop.fo.flow.ExternalGraphic;
-import org.apache.fop.fo.flow.Float;
 import org.apache.fop.fo.flow.Footnote;
 import org.apache.fop.fo.flow.FootnoteBody;
-import org.apache.fop.fo.flow.InitialPropertySet;
 import org.apache.fop.fo.flow.Inline;
 import org.apache.fop.fo.flow.InlineContainer;
 import org.apache.fop.fo.flow.InstreamForeignObject;
@@ -76,18 +69,12 @@ import org.apache.fop.fo.flow.ListItem;
 import org.apache.fop.fo.flow.ListItemBody;
 import org.apache.fop.fo.flow.ListItemLabel;
 import org.apache.fop.fo.flow.Marker;
-import org.apache.fop.fo.flow.MultiCase;
-import org.apache.fop.fo.flow.MultiProperties;
-import org.apache.fop.fo.flow.MultiPropertySet;
-import org.apache.fop.fo.flow.MultiSwitch;
-import org.apache.fop.fo.flow.MultiToggle;
 import org.apache.fop.fo.flow.PageNumber;
 import org.apache.fop.fo.flow.PageNumberCitation;
 import org.apache.fop.fo.flow.RetrieveMarker;
 import org.apache.fop.fo.flow.Table;
 import org.apache.fop.fo.flow.TableAndCaption;
 import org.apache.fop.fo.flow.TableBody;
-import org.apache.fop.fo.flow.TableCaption;
 import org.apache.fop.fo.flow.TableCell;
 import org.apache.fop.fo.flow.TableColumn;
 import org.apache.fop.fo.flow.TableFooter;
@@ -174,6 +161,19 @@ public class AddLMVisitor {
         return saveLMList;
     }
 
+    /**
+     * @param node FONode object to process
+     */
+    public void serveFONode(FONode node) {
+    }
+
+    /**
+     * @param node FObj object to process
+     */
+    public void serveFObj(FObj node) {
+        serveFONode((FONode)node);
+    }
+
     public void serveFOText(FOText foText) {
         if (foText.endIndex - foText.startIndex > 0) {
             currentLMList.add(new TextLayoutManager(foText));
@@ -798,18 +798,18 @@ public class AddLMVisitor {
      }
 
     /**
-     * @param node FONode object to process
-     */
-    public void serveFONode(FONode node) {
-    }
-
-    /**
-     * @param node FObj object to process
+     * @param node Wrapper object to process
      */
-    public void serveFObj(FObj node) {
-        serveFONode((FONode)node);
+    public void serveWrapper(Wrapper node) {
+        ListIterator baseIter;
+        baseIter = node.getChildNodes();
+        if (baseIter == null) return;
+        while (baseIter.hasNext()) {
+            FObj child = (FObj) baseIter.next();
+            child.acceptVisitor(this);
+        }
     }
-
+    
     /**
      * @param node ColorProfile object to process
      */
@@ -880,19 +880,6 @@ public class AddLMVisitor {
         serveFObjMixed((FObjMixed)node);
     }
 
-    /**
-     * @param node Wrapper object to process
-     */
-    public void serveWrapper(Wrapper node) {
-        ListIterator baseIter;
-        baseIter = node.getChildNodes();
-        if (baseIter == null) return;
-        while (baseIter.hasNext()) {
-            FObj child = (FObj) baseIter.next();
-            child.acceptVisitor(this);
-        }
-    }
-    
     /**
      * @param node FootnoteBody object to process
      */
@@ -1053,117 +1040,4 @@ public class AddLMVisitor {
     public void serveTableColumn(TableColumn node) {
         serveFObj((FObj)node);
     }
-
-    /**
-     * @param node ToBeImplementedElement object to process
-     */
-    public void serveToBeImplementedElement(ToBeImplementedElement node) {
-        serveFObj((FObj)node);
-    }
-
-    /**
-     * @param node Float object to process
-     */
-    public void serveFloat(Float node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node InitialPropertySet object to process
-     */
-    public void serveInitialPropertySet(InitialPropertySet node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node MultiCase object to process
-     */
-    public void serveMultiCase(MultiCase node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node MultiProperties object to process
-     */
-    public void serveMultiProperties(MultiProperties node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node MultiPropertySet object to process
-     */
-    public void serveMultiPropertySet(MultiPropertySet node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node MultiSwitch object to process
-     */
-    public void serveMultiSwitch(MultiSwitch node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node MultiToggle object to process
-     */
-    public void serveMultiToggle(MultiToggle node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node TableAndCaption object to process
-     */
-    public void serveTableAndCaption(TableAndCaption node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node TableCaption object to process
-     */
-    public void serveTableCaption(TableCaption node) {
-        serveToBeImplementedElement((ToBeImplementedElement)node);
-    }
-
-    /**
-     * @param node Unknown object to process
-     */
-    public void serveUnknown(Unknown node) {
-        serveFONode((FONode)node);
-    }
-
-    /**
-     * @param node XMLObj object to process
-     */
-    public void serveXMLObj(XMLObj node) {
-        serveFONode((FONode)node);
-    }
-
-    /**
-     * @param node SVGObj object to process
-     */
-    public void serveSVGObj(SVGObj node) {
-        serveXMLObj((XMLObj)node);
-    }
-
-    /**
-     * @param node SVGElement object to process
-     */
-    public void serveSVGElement(SVGElement node) {
-        serveSVGObj((SVGObj)node);
-    }
-
-    /**
-     * @param node UnknownXMLObj object to process
-     */
-    public void serveUnknownXMLObj(UnknownXMLObj node) {
-        serveXMLObj((XMLObj)node);
-    }
-
-    /**
-     * @param node XMLElement object to process
-     */
-    public void serveXMLElement(XMLElement node) {
-        serveXMLObj((XMLObj)node);
-    }
-
 }