Renamed FONode.addChild() to somewhat clearer FONode.addChildNode().

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

diff --git a/src/java/org/apache/fop/fo/FONode.java b/src/java/org/apache/fop/fo/FONode.java
index c3927f9b6fb3af04cba9e317c73724b07bd93d21..8314084d58ef3af29a7e17dff6282cae4f195657 100644 (file)
--- a/src/java/org/apache/fop/fo/FONode.java
+++ b/src/java/org/apache/fop/fo/FONode.java
@@ -167,7 +167,7 @@ public abstract class FONode {
     /**
      * @param child child node to be added to the children of this node
      */
-    protected void addChild(FONode child) {
+    protected void addChildNode(FONode child) {
     }
 
     /**

diff --git a/src/java/org/apache/fop/fo/FOTreeBuilder.java b/src/java/org/apache/fop/fo/FOTreeBuilder.java
index 15cbc3b06b3ece5091ae584a5a4064cc2a4ada80..faef21b62b56bc703ce246e46626a2842d97d127 100644 (file)
--- a/src/java/org/apache/fop/fo/FOTreeBuilder.java
+++ b/src/java/org/apache/fop/fo/FOTreeBuilder.java
@@ -276,7 +276,7 @@ public class FOTreeBuilder extends DefaultHandler {
             rootFObj = (Root) foNode;
             rootFObj.setFOInputHandler(foInputHandler);
         } else {
-            currentFObj.addChild(foNode);
+            currentFObj.addChildNode(foNode);
         }
 
         currentFObj = foNode;

diff --git a/src/java/org/apache/fop/fo/FObj.java b/src/java/org/apache/fop/fo/FObj.java
index 06dd20a5b0721fcb937cb3a21958bd6effe3c9b6..b043ce75ab38ad8b40d748745daec630a9677249 100644 (file)
--- a/src/java/org/apache/fop/fo/FObj.java
+++ b/src/java/org/apache/fop/fo/FObj.java
@@ -251,11 +251,9 @@ public class FObj extends FONode implements Constants {
     }
 
     /**
-     * Add the child to this object.
-     *
-     * @param child the child node to add
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode child) {
+    protected void addChildNode(FONode child) {
         if (containsMarkers() && "fo:marker".equals(child.getName())) {
             addMarker((Marker)child);
         } else {

diff --git a/src/java/org/apache/fop/fo/FObjMixed.java b/src/java/org/apache/fop/fo/FObjMixed.java
index 18ddcf4e4b6e34faf3e114462462333977b6d7a9..e93eef37aca747b9e8c2a1780ea5a0d28c453ecc 100644 (file)
--- a/src/java/org/apache/fop/fo/FObjMixed.java
+++ b/src/java/org/apache/fop/fo/FObjMixed.java
@@ -57,7 +57,7 @@ public class FObjMixed extends FObj {
         /* characters() processing empty for FOTreeHandler, not empty for RTF & MIFHandlers */
         getFOInputHandler().characters(ft.ca, ft.startIndex, ft.endIndex);
 
-        addChild(ft);
+        addChildNode(ft);
     }
 
     private void setup() {

diff --git a/src/java/org/apache/fop/fo/UnknownXMLObj.java b/src/java/org/apache/fop/fo/UnknownXMLObj.java
index 95750b8230cd1f2623a94bb25c93b614fcd7f820..4fcd6c081e7b62a7c3b9670e21dd8f30474806d5 100644 (file)
--- a/src/java/org/apache/fop/fo/UnknownXMLObj.java
+++ b/src/java/org/apache/fop/fo/UnknownXMLObj.java
@@ -69,13 +69,13 @@ public class UnknownXMLObj extends XMLObj {
     }
 
     /**
-     * @see XMLObj#addChild
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode child) {
+    protected void addChildNode(FONode child) {
         if (doc == null) {
             createBasicDocument();
         }
-        super.addChild(child);
+        super.addChildNode(child);
     }
 
     /**

diff --git a/src/java/org/apache/fop/fo/XMLObj.java b/src/java/org/apache/fop/fo/XMLObj.java
index 9d6b7f4cf45a6564ca33345d4d33b9e06f95374d..d6c177b2208b92f1a94fb9441d8390438bfc60ba 100644 (file)
--- a/src/java/org/apache/fop/fo/XMLObj.java
+++ b/src/java/org/apache/fop/fo/XMLObj.java
@@ -170,9 +170,9 @@ public abstract class XMLObj extends FONode {
     }
 
     /**
-     * @param child FONode child that should be added to this node
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode child) {
+    protected void addChildNode(FONode child) {
         if (child instanceof XMLObj) {
             ((XMLObj)child).addElement(doc, element);
         } else {

diff --git a/src/java/org/apache/fop/fo/extensions/Bookmarks.java b/src/java/org/apache/fop/fo/extensions/Bookmarks.java
index e7d45f50819376dddb5405e6ba33ff4fd8efd436..8405f4b29acd166f3749efe89b1f264549c11be9 100644 (file)
--- a/src/java/org/apache/fop/fo/extensions/Bookmarks.java
+++ b/src/java/org/apache/fop/fo/extensions/Bookmarks.java
@@ -44,12 +44,9 @@ public class Bookmarks extends ExtensionObj {
     }
 
     /**
-     * Add the child to the top level.
-     * This handles all Outline objects added and ignores others.
-     *
-     * @param obj the child to add
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode obj) {
+    protected void addChildNode(FONode obj) {
         if (obj instanceof Outline) {
             outlines.add(obj);
         }

diff --git a/src/java/org/apache/fop/fo/extensions/Outline.java b/src/java/org/apache/fop/fo/extensions/Outline.java
index c6f6194f86a876a2f994c05e3ec6aaecbf2f43d8..725e5ebe6ecbe872215d8a3df6286950c1ccfa63 100644 (file)
--- a/src/java/org/apache/fop/fo/extensions/Outline.java
+++ b/src/java/org/apache/fop/fo/extensions/Outline.java
@@ -69,12 +69,9 @@ public class Outline extends ExtensionObj {
     }
 
     /**
-     * Add the child to this outline.
-     * This checks for the type, label or outline and handles appropriately.
-     *
-     * @param obj the child object
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode obj) {
+    protected void addChildNode(FONode obj) {
         if (obj instanceof Label) {
             label = (Label)obj;
         } else if (obj instanceof Outline) {

diff --git a/src/java/org/apache/fop/fo/flow/Block.java b/src/java/org/apache/fop/fo/flow/Block.java
index f7f57ab54ba12d1a068cdd342dc0427b28e5ab10..c9e05e192ca47a9818eef35ec7c815056bf45310 100644 (file)
--- a/src/java/org/apache/fop/fo/flow/Block.java
+++ b/src/java/org/apache/fop/fo/flow/Block.java
@@ -216,9 +216,9 @@ public class Block extends FObjMixed {
     }
 
     /**
-     * @see org.apache.fop.fo.FObj#addChild
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    public void addChild(FONode child) {
+    public void addChildNode(FONode child) {
         // Handle whitespace based on values of properties
         // Handle a sequence of inline-producing children in
         // one pass
@@ -231,7 +231,7 @@ public class Block extends FObjMixed {
             // Handle whitespace in preceeding inline areas if any
             handleWhiteSpace();
         }
-        super.addChild(child);
+        super.addChildNode(child);
     }
 
     /**

diff --git a/src/java/org/apache/fop/fo/flow/Footnote.java b/src/java/org/apache/fop/fo/flow/Footnote.java
index 83a50e50a48b58d4b96f965782785b3953c700da..612668bb431aca397790bdfdab4c9e4b19f28a21 100644 (file)
--- a/src/java/org/apache/fop/fo/flow/Footnote.java
+++ b/src/java/org/apache/fop/fo/flow/Footnote.java
@@ -53,9 +53,9 @@ public class Footnote extends FObj {
     }
 
     /**
-     * @param child child FONode to be added to this object
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    public void addChild(FONode child) {
+    public void addChildNode(FONode child) {
         String name = child.getName();
         if ("fo:inline".equals(name)) {
             inlineFO = (Inline)child;

diff --git a/src/java/org/apache/fop/fo/flow/ListItem.java b/src/java/org/apache/fop/fo/flow/ListItem.java
index 5298fa99deb0ce4f58aef5fbbde04a62f9fa87b9..ae994c1a884aee66118e4373d7f52f630ac01043 100644 (file)
--- a/src/java/org/apache/fop/fo/flow/ListItem.java
+++ b/src/java/org/apache/fop/fo/flow/ListItem.java
@@ -105,10 +105,9 @@ public class ListItem extends FObj {
     }
 
     /**
-     * Overridden from FObj
-     * @param child the child object to be added
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    public void addChild(FONode child) {
+    public void addChildNode(FONode child) {
         if ("fo:list-item-label".equals(child.getName())) {
             label = (ListItemLabel)child;
         } else if ("fo:list-item-body".equals(child.getName())) {

diff --git a/src/java/org/apache/fop/fo/flow/Table.java b/src/java/org/apache/fop/fo/flow/Table.java
index aa98f304b5adee7312149d83736b7adc01004209..5f09d8539418e770da20c799cd2409bd990bce08 100644 (file)
--- a/src/java/org/apache/fop/fo/flow/Table.java
+++ b/src/java/org/apache/fop/fo/flow/Table.java
@@ -85,10 +85,9 @@ public class Table extends FObj {
     }
 
     /**
-     * Overrides FObj.
-     * @param child FONode child object to be added
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode child) {
+    protected void addChildNode(FONode child) {
         if (child.getName().equals("fo:table-column")) {
             if (columns == null) {
                 columns = new ArrayList();
@@ -100,7 +99,7 @@ public class Table extends FObj {
             tableHeader = (TableBody)child;
         } else {
             // add bodies
-            super.addChild(child);
+            super.addChildNode(child);
         }
     }
 

diff --git a/src/java/org/apache/fop/fo/pagination/PageSequence.java b/src/java/org/apache/fop/fo/pagination/PageSequence.java
index 33e8e044ae0b2957731679acf6bb5ca1fc225aab..5dd7c981503c540800dc825f6916e51ff9054758 100644 (file)
--- a/src/java/org/apache/fop/fo/pagination/PageSequence.java
+++ b/src/java/org/apache/fop/fo/pagination/PageSequence.java
@@ -180,13 +180,9 @@ public class PageSequence extends FObj {
     }
 
     /**
-     * Validate the child being added and initialize internal variables.
-     * XSL content model for page-sequence:
-     * <pre>(title?,static-content*,flow)</pre>
-     *
-     * @param child The flow object child to be added to the PageSequence.
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    public void addChild(FONode child) {
+    public void addChildNode(FONode child) {
         try {
             String childName = child.getName();
             if (childName.equals("fo:title")) {
@@ -207,7 +203,7 @@ public class PageSequence extends FObj {
                 // Don't add main flow to the flow map
 //              addFlow(mainFlow);
                 startStructuredPageSequence();
-                super.addChild(child); // For getChildren
+                super.addChildNode(child); // For getChildren
             } else if (childName.equals("fo:static-content")) {
                 String flowName = ((StaticContent)child).getFlowName();
                 if (flowMap.containsKey(flowName)) {
@@ -223,7 +219,7 @@ public class PageSequence extends FObj {
                 startStructuredPageSequence();
             } 
         } catch (FOPException fopex) {
-            getLogger().error("Error in PageSequence.addChild(): "
+            getLogger().error("Error in PageSequence.addChildNode(): "
                 + fopex.getMessage(), fopex);
         }
     }

diff --git a/src/java/org/apache/fop/fo/pagination/SimplePageMaster.java b/src/java/org/apache/fop/fo/pagination/SimplePageMaster.java
index 1fbdb06208d7b9ad5ad47caaf3e1425ab79d4413..5a089637e6839aab5360d3809f79118ff234b5a8 100644 (file)
--- a/src/java/org/apache/fop/fo/pagination/SimplePageMaster.java
+++ b/src/java/org/apache/fop/fo/pagination/SimplePageMaster.java
@@ -92,9 +92,9 @@ public class SimplePageMaster extends FObj {
     }
 
     /**
-     * @see org.apache.fop.fo.FONode#addChild(FONode)
+     * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
-    protected void addChild(FONode child) {
+    protected void addChildNode(FONode child) {
         if (child instanceof Region) {
             addRegion((Region)child);
         } else {