]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Second phase of performance improvement.
authorFinn Bock <bckfnn@apache.org>
Tue, 19 Oct 2004 13:45:38 +0000 (13:45 +0000)
committerFinn Bock <bckfnn@apache.org>
Tue, 19 Oct 2004 13:45:38 +0000 (13:45 +0000)
- Removed addProperties() method and misc old property fields.

PR: 31699

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

40 files changed:
src/java/org/apache/fop/fo/flow/BasicLink.java
src/java/org/apache/fop/fo/flow/BidiOverride.java
src/java/org/apache/fop/fo/flow/Block.java
src/java/org/apache/fop/fo/flow/BlockContainer.java
src/java/org/apache/fop/fo/flow/Character.java
src/java/org/apache/fop/fo/flow/ExternalGraphic.java
src/java/org/apache/fop/fo/flow/Footnote.java
src/java/org/apache/fop/fo/flow/FootnoteBody.java
src/java/org/apache/fop/fo/flow/Inline.java
src/java/org/apache/fop/fo/flow/InlineContainer.java
src/java/org/apache/fop/fo/flow/Leader.java
src/java/org/apache/fop/fo/flow/ListBlock.java
src/java/org/apache/fop/fo/flow/ListItem.java
src/java/org/apache/fop/fo/flow/ListItemBody.java
src/java/org/apache/fop/fo/flow/ListItemLabel.java
src/java/org/apache/fop/fo/flow/Marker.java
src/java/org/apache/fop/fo/flow/PageNumber.java
src/java/org/apache/fop/fo/flow/PageNumberCitation.java
src/java/org/apache/fop/fo/flow/RetrieveMarker.java
src/java/org/apache/fop/fo/flow/Table.java
src/java/org/apache/fop/fo/flow/TableBody.java
src/java/org/apache/fop/fo/flow/TableCell.java
src/java/org/apache/fop/fo/flow/TableColumn.java
src/java/org/apache/fop/fo/flow/TableRow.java
src/java/org/apache/fop/fo/pagination/ConditionalPageMasterReference.java
src/java/org/apache/fop/fo/pagination/Flow.java
src/java/org/apache/fop/fo/pagination/LayoutMasterSet.java
src/java/org/apache/fop/fo/pagination/PageSequence.java
src/java/org/apache/fop/fo/pagination/PageSequenceMaster.java
src/java/org/apache/fop/fo/pagination/Region.java
src/java/org/apache/fop/fo/pagination/RegionAfter.java
src/java/org/apache/fop/fo/pagination/RegionBA.java
src/java/org/apache/fop/fo/pagination/RegionBefore.java
src/java/org/apache/fop/fo/pagination/RegionEnd.java
src/java/org/apache/fop/fo/pagination/RegionSE.java
src/java/org/apache/fop/fo/pagination/RegionStart.java
src/java/org/apache/fop/fo/pagination/RepeatablePageMasterAlternatives.java
src/java/org/apache/fop/fo/pagination/RepeatablePageMasterReference.java
src/java/org/apache/fop/fo/pagination/SimplePageMaster.java
src/java/org/apache/fop/fo/pagination/SinglePageMasterReference.java

index 3b9424b99b9a36fd995c98e7611d208355bb76e8..ccb94584a25bd96dbce18334311b9d4a469e8a96 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -51,12 +50,6 @@ public class BasicLink extends Inline {
     // private ToBeImplementedProperty targetStylesheet;
     // End of property values
 
-    // link represented by this FO
-    private String link = null;
-    
-    // indicator of whether link is internal or external
-    private boolean isExternalLink = false;
-
     // used only for FO validation
     private boolean blockOrInlineItemFound = false;
 
@@ -108,31 +101,6 @@ public class BasicLink extends Inline {
         getFOEventHandler().endLink();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        
-        // This logic is for determining the link represented by this FO.
-        String ext =  propertyList.get(PR_EXTERNAL_DESTINATION).getString();
-        String internal = propertyList.get(PR_INTERNAL_DESTINATION).getString();
-
-        // per spec, internal takes precedence if both specified        
-        if (internal.length() > 0) { 
-            link = internal;
-        } else if (ext.length() > 0) {
-            link = ext;
-            isExternalLink = true;
-        } else {
-            // slightly stronger than spec "should be specified"
-            attributeError("Missing attribute:  Either external-destination or " +
-                "internal-destination must be specified.");
-        }
-        
-        getFOEventHandler().startLink(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (#PCDATA|%inline;|%block;)*
@@ -158,20 +126,6 @@ public class BasicLink extends Inline {
         list.add(lm);
     }
 
-    /**
-     * @return link represented by this fo:basic-link
-     */
-    public String getLink() {
-        return link;
-    }
-    /**
-     * @return true if link is external, false if internal
-     */
-    public boolean isExternalLink() {
-        return isExternalLink;
-    }
-
     /**
      * Return the "internal-destination" property.
      */
index 6d15a8ebca99fc8250936e147c239123accee853..3f5740a35456e50af211862c39410d356de680e4 100644 (file)
@@ -23,7 +23,6 @@ import java.util.ArrayList;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -107,15 +106,6 @@ public class BidiOverride extends FObjMixed {
         prWordSpacing = pList.get(PR_WORD_SPACING).getSpace();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     * @todo see if can use a BitSet to determine if an FO should
-     * have its ID setup; then move setupID() instances to FObj.
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (#PCDATA|%inline;|%block;)*
index 9ae7963ea39304066d0ac4df58100ba9953c9c3e..b7c54bb7954904ec3ee318eb93edb75418c1b651 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -110,23 +109,6 @@ public class Block extends FObjMixed {
     private int wrapOption;
     // End of property values
     
-    private int align;
-    private int alignLast;
-    private int _lineHeight;
-    private int startIndent;
-    private int endIndent;
-    private int spaceBefore;
-    private int spaceAfter;
-    private int _textIndent;
-    private int _keepWithNext;
-    private ColorType backgroundColor;
-    private int blockWidows;
-    private int blockOrphans;
-
-    private int wsTreatment; //ENUMERATION
-    private int lfTreatment; //ENUMERATION
-    private boolean bWScollapse; //true if white-space-collapse=true
-
     // this may be helpful on other FOs too
     private boolean anythingLaidOut = false;
 
@@ -292,33 +274,6 @@ public class Block extends FObjMixed {
         return textIndent;
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.span = getPropEnum(PR_SPAN);
-        this.wsTreatment = getPropEnum(PR_WHITE_SPACE_TREATMENT);
-        this.bWScollapse = (getPropEnum(PR_WHITE_SPACE_COLLAPSE) == Constants.TRUE);
-        this.lfTreatment = getPropEnum(PR_LINEFEED_TREATMENT);
-        this.align = getPropEnum(PR_TEXT_ALIGN);
-        this.alignLast = getPropEnum(PR_TEXT_ALIGN_LAST);
-        this.breakAfter = getPropEnum(PR_BREAK_AFTER);
-        this._lineHeight = getPropLength(PR_LINE_HEIGHT);
-        this.startIndent = getPropLength(PR_START_INDENT);
-        this.endIndent = getPropLength(PR_END_INDENT);
-        this.spaceBefore = getPropLength(PR_SPACE_BEFORE | CP_OPTIMUM);
-        this.spaceAfter = getPropLength(PR_SPACE_AFTER | CP_OPTIMUM);
-        this._textIndent = getPropLength(PR_TEXT_INDENT);
-        this._keepWithNext = getPropEnum(PR_KEEP_WITH_NEXT);
-        this.blockWidows =
-          this.propertyList.get(PR_WIDOWS).getNumber().intValue();
-        this.blockOrphans =
-          this.propertyList.get(PR_ORPHANS).getNumber().intValue();
-
-        getFOEventHandler().startBlock(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* initial-property-set? (#PCDATA|%inline;|%block;)*
@@ -391,7 +346,7 @@ public class Block extends FObjMixed {
                         /* Some kind of whitespace character, except linefeed. */
                         boolean bIgnore = false;
 
-                        switch (wsTreatment) {
+                        switch (whiteSpaceTreatment) {
                             case Constants.IGNORE:
                                 bIgnore = true;
                                 break;
@@ -412,8 +367,8 @@ public class Block extends FObjMixed {
                         // Handle ignore and replacement
                         if (bIgnore) {
                             charIter.remove();
-                        } else if (bWScollapse) {
-                            if (bInWS || (lfTreatment == Constants.PRESERVE
+                        } else if (whiteSpaceCollapse == TRUE) {
+                            if (bInWS || (linefeedTreatment == Constants.PRESERVE
                                         && (bPrevWasLF || lfCheck.nextIsLF()))) {
                                 charIter.remove();
                             } else {
@@ -442,7 +397,7 @@ public class Block extends FObjMixed {
                         lfCheck.reset();
                         bPrevWasLF = true; // for following whitespace
 
-                        switch (lfTreatment) {
+                        switch (linefeedTreatment) {
                             case Constants.IGNORE:
                                 charIter.remove();
                                 break;
@@ -451,7 +406,7 @@ public class Block extends FObjMixed {
                                     // only if bWScollapse=true
                                     charIter.remove();
                                 } else {
-                                    if (bWScollapse) {
+                                    if (whiteSpaceCollapse == TRUE) {
                                         bInWS = true;
                                         // remove the linefeed if no word in block 
                                         // encountered yet
index df1a8e884f7e2c5fa1411022c6c00907d7e21d5c..643d7577be3381ca8f6f3c42ed04a4b9931ea18c 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.datatypes.Length;
 import org.apache.fop.datatypes.Numeric;
 import org.apache.fop.fo.FONode;
@@ -35,7 +34,6 @@ import org.apache.fop.fo.properties.KeepProperty;
 import org.apache.fop.fo.properties.LengthRangeProperty;
 import org.apache.fop.layoutmgr.BlockContainerLayoutManager;
 
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 /**
@@ -67,17 +65,6 @@ public class BlockContainer extends FObj {
     // private ToBeImplementedProperty zIndex;
     // End of property values
 
-    private ColorType backgroundColor;
-    private int position;
-
-    private int top;
-    private int bottom;
-    private int left;
-    private int right;
-    private int _width;
-    private int _height;
-
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -120,20 +107,6 @@ public class BlockContainer extends FObj {
         getFOEventHandler().startBlockContainer(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.span = getPropEnum(PR_SPAN);
-        this.backgroundColor =
-            this.propertyList.get(PR_BACKGROUND_COLOR).getColorType();
-
-        this._width = getPropLength(PR_WIDTH);
-        this._height = getPropLength(PR_HEIGHT);
-        getFOEventHandler().startBlockContainer(this);
-    }
-    
     /**
      * @see org.apache.fop.fo.FONode#endOfNode
      */
index 2a84a6dcfc1ec9f232d52d517478d9d87768c17b..464ae434a0b5b1775386c3fd70e17916c94ad15a 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -95,8 +94,6 @@ public class Character extends FObj {
     /** constant indicating that the character does not fit */
     public static final int DOESNOT_FIT = 1;
 
-    private char characterValue;
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -157,21 +154,11 @@ public class Character extends FObj {
             invalidChildError(loc, nsURI, localName);
     }
 
-
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        characterValue = propertyList.get(PR_CHARACTER).getCharacter();
-        getFOEventHandler().character(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FObj#charIterator
      */
     public CharIterator charIterator() {
-        return new OneCharIterator(characterValue);
+        return new OneCharIterator(character);
         // But what if the character is ignored due to white space handling?
     }
 
index 290183b51e4934fd94f98d2584f31566d2a5aa68..344886140d75c911d420ec7ea2b28a1f8e7bdea3 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -137,15 +136,6 @@ public class ExternalGraphic extends FObj {
             invalidChildError(loc, nsURI, localName);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     * @todo switch method from image() to startImage()?
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().image(this);
-    }
-
     /**
      * Return the Common Border, Padding, and Background Properties.
      */
index 1b5506e2d89eccb6e3fa199a7b521f0c6e98cad6..7c7bc1a91319b2a30d519e73ce22ed43f5cef4e3 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -57,14 +56,6 @@ public class Footnote extends FObj {
         commonAccessibility = pList.getAccessibilityProps();
     }
     
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().startFootnote(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#startOfNode
      */
index d582d9e1b532526337d2a6ca940ee6afe72e8ba9..671e44d00f1f1790da73dad57dbdc5cd782c8fa8 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.flow;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -70,14 +69,6 @@ public class FootnoteBody extends FObj {
         getFOEventHandler().startFootnoteBody(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().startFootnoteBody(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: (%block;)+
index a1ae503005d7492714b02f05814f4c1ed7d52c1d..ce88a2f8b96ab23adc3d026adfadef25b6aba383 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.flow;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -143,33 +142,6 @@ public class Inline extends FObjMixed {
         getFOEventHandler().endInline(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-       /* Check to see if this node can have block-level children.
-        * See validateChildNode() below.
-        */
-       int lvlLeader = findAncestor(FO_LEADER);
-       int lvlFootnote = findAncestor(FO_FOOTNOTE);
-       int lvlInCntr = findAncestor(FO_INLINE_CONTAINER);
-
-       if (lvlLeader > 0) {
-           if (lvlInCntr < 0 ||
-               (lvlInCntr > 0 && lvlInCntr > lvlLeader)) {
-               canHaveBlockLevelChildren = false;
-           }
-       } else if (lvlFootnote > 0) {
-           if (lvlInCntr < 0 || lvlInCntr > lvlFootnote) {
-               canHaveBlockLevelChildren = false;
-           }
-       }
-
-        getFOEventHandler().startInline(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (#PCDATA|%inline;|%block;)*
index 5f7d22e5d9a16e4c85ae3f70c2d12fd7f6abd411..7fae66003d36d4c571b243ff91ad500179d21530 100644 (file)
@@ -23,7 +23,6 @@ import java.util.List;
 import java.util.ArrayList;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 // FOP
@@ -110,13 +109,6 @@ public class InlineContainer extends FObj {
         checkId(id);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-    }
-
     /**
      * Return the "id" property.
      */
index d5752907ac5f88ab4f36ea031202dfbb400889ba..87cd555ec80b83a2c458733c01415271c37c53f3 100644 (file)
@@ -80,11 +80,6 @@ public class Leader extends FObjMixed {
     private SpaceProperty wordSpacing;
     // End of property values
 
-    private int _ruleThickness;
-    private int patternWidth;
-    /** FontState for this object */
-    protected Font fontState;
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -146,51 +141,12 @@ public class Leader extends FObjMixed {
     protected void startOfNode() throws SAXParseException {
         checkId(id);
     }
-    
-    /**
-     * @todo convert to addProperties()
-     */
-    private void setup() {
-        // Common Font Properties
-        this.fontState = propMgr.getFontState(getFOEventHandler().getFontInfo());
-
-        // color properties
-        ColorType c = this.propertyList.get(PR_COLOR).getColorType();
-        float red = c.getRed();
-        float green = c.getGreen();
-        float blue = c.getBlue();
-
-        // fo:leader specific properties
-        // determines the pattern of leader; allowed values: space, rule,dots, use-content
-        leaderPattern = getPropEnum(PR_LEADER_PATTERN);
-        switch(leaderPattern) {
-            case LeaderPattern.SPACE:
-                // use Space
-            break;
-            case LeaderPattern.RULE:
-                // the following properties only apply
-                // for leader-pattern = "rule"
-                _ruleThickness = getPropLength(PR_RULE_THICKNESS);
-                ruleStyle = getPropEnum(PR_RULE_STYLE);
-            break;
-            case LeaderPattern.DOTS:
-            break;
-            case LeaderPattern.USECONTENT:
-                // use inline layout manager to create inline areas
-                // add the inline parent multiple times until leader full
-            break;
-        }
-
-        // if leaderPatternWidth = 0 = default = use-font-metric
-        patternWidth = getPropLength(PR_LEADER_PATTERN_WIDTH);
-    }
 
     /**
      * @todo check need for each of these accessors (should be LM instead?)
      */
-    public int getLength(int propId, int dim) {
+    public int getLength(Length maxlength, int dim) {
         int length;
-        Length maxlength = propertyList.get(propId).getLength();
         if (maxlength instanceof PercentLength) {
             length = (int)(((PercentLength)maxlength).value() * dim);
         } else {
@@ -200,15 +156,15 @@ public class Leader extends FObjMixed {
     }
 
     public int getRuleThickness() {
-        return _ruleThickness;
+        return ruleThickness.getValue();
     }
 
     public Font getFontState() {
-        return fontState;
+        return propMgr.getFontState(getFOEventHandler().getFontInfo());
     }
 
     public int getPatternWidth() {
-        return patternWidth;
+        return leaderPatternWidth.getValue();
     }
 
     /**
@@ -272,7 +228,6 @@ public class Leader extends FObjMixed {
      * @see org.apache.fop.fo.FONode#addLayoutManager(List)
      */
     public void addLayoutManager(List list) {
-        setup();
         LeaderLayoutManager lm = new LeaderLayoutManager(this);
         list.add(lm);
     }
index 6c99580aa5f89ed7309818a9673eec2798b54676..75c86be63a3dc7a9a62e641409a992376d70e6d0 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -110,14 +109,6 @@ public class ListBlock extends FObj {
         getFOEventHandler().endList(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().startList(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (list-item)+
index 6b04f22e1339c0c99ed390c4275a3e43829d49a2..6297a619fa7c9b7dc49f3ff4a481bc856419fd73 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -107,14 +106,6 @@ public class ListItem extends FObj {
         getFOEventHandler().endListItem(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().startListItem(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (list-item-label,list-item-body)
index 2b936f68ac0f22ff317b7c9dc9abfd65731a1115..1f79c758448c416a8d181b4a78668cb7f432717b 100644 (file)
@@ -70,18 +70,6 @@ public class ListItemBody extends FObj {
         getFOEventHandler().endListBody();
     }
 
-    /**
-     * @todo convert to addProperties()
-     */
-    private void setup() {
-        /*
-         * For calculating the lineage - The fo:list-item-body formatting object
-         * does not generate any areas. The fo:list-item-body formatting object
-         * returns the sequence of areas created by concatenating the sequences
-         * of areas returned by each of the child nodes of the fo:list-item-body.
-         */
-    }
-
     /**
      * Return the "id" property.
      */
index f7e3ec06380e683e1a8b41a6f875c1aebf94171f..3f02ca7f2ce77a3ffb132cdc2dca7a5f11f51109 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.flow;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 // FOP
@@ -71,20 +70,6 @@ public class ListItemLabel extends FObj {
         getFOEventHandler().endListLabel();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().startListLabel();
-        /*
-         * For calculating the lineage - The fo:list-item-label formatting object
-         * does not generate any areas. The fo:list-item-label formatting object
-         * returns the sequence of areas created by concatenating the sequences
-         * of areas returned by each of the child nodes of the fo:list-item-label.
-         */
-    }
-
     /**
      * Return the "id" property.
      */
index 066d40189b62fc2743853eb0204a62972838bfef..68a473a7d003e768362c447937bf4b98c0275d22 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.flow;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -51,13 +50,6 @@ public class Marker extends FObjMixed {
         markerClassName = pList.get(PR_MARKER_CLASS_NAME).getString();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: (#PCDATA|%inline;|%block;)*
index 5671cddb271494cf5ece3a433d14b44917a7fc71..a48e65e20cf1be9c267ca43b40542d87de5ab4aa 100644 (file)
@@ -22,12 +22,10 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.datatypes.Length;
 import org.apache.fop.fo.FONode;
 import org.apache.fop.fo.FObj;
@@ -74,13 +72,6 @@ public class PageNumber extends FObj {
     private int wrapOption;
     //  End of property values
 
-    /** FontState for this object */
-    protected Font fontState;
-
-    private float red;
-    private float green;
-    private float blue;
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -142,29 +133,12 @@ public class PageNumber extends FObj {
             invalidChildError(loc, nsURI, localName);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        // Common Font Properties
-        this.fontState = propMgr.getFontState(getFOEventHandler().getFontInfo());
-
-        ColorType c = this.propertyList.get(PR_COLOR).getColorType();
-        this.red = c.getRed();
-        this.green = c.getGreen();
-        this.blue = c.getBlue();
-
-        this.wrapOption = getPropEnum(PR_WRAP_OPTION);
-        getFOEventHandler().startPageNumber(this);
-    }
-
     /**
      * @return the FontState object for this PageNumber
      */
     public Font getFontState() {
-        return fontState;
+        return propMgr.getFontState(getFOEventHandler().getFontInfo());
+
     }
 
     /**
index 89e2b6127063f5983fbd3af7b1647d219cabe37a..9d62c96db8738c3b92609fb97bfbab86f553f28a 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -133,17 +132,6 @@ public class PageNumberCitation extends FObj {
             invalidChildError(loc, nsURI, localName);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        if (getPropString(PR_REF_ID) == null || getPropString(PR_REF_ID).equals("")) {
-            missingPropertyError("ref-id");
-        }
-    }
-
     /**
      * Return the Common Font Properties.
      */
index a944b69bd959d73992dc12dc56348edbd8392405..aff9b0e02e4ea4a894e9bbc2798dba461e771e3f 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -63,16 +62,6 @@ public class RetrieveMarker extends FObjMixed {
         retrieveBoundary = pList.get(PR_RETRIEVE_BOUNDARY).getEnum();
     }
     
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.retrieveClassName = getPropString(PR_RETRIEVE_CLASS_NAME);
-        this.retrievePosition = getPropEnum(PR_RETRIEVE_POSITION);
-        this.retrieveBoundary = getPropEnum(PR_RETRIEVE_BOUNDARY);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: empty
index b9457a2e4cb8852083e29bfe34bee246a9b8dc88..9ebd94d5f491fd86eb5c8075ab72dc3191bf298b 100644 (file)
@@ -24,11 +24,9 @@ import java.util.ListIterator;
 import java.util.ArrayList;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.datatypes.Length;
 import org.apache.fop.fo.FONode;
 import org.apache.fop.fo.FObj;
@@ -84,23 +82,6 @@ public class Table extends FObj {
     protected ArrayList columns = null;
     private TableBody tableHeader = null;
     private TableBody tableFooter = null;
-    private boolean omitHeaderAtBreak = false;
-    private boolean omitFooterAtBreak = false;
-
-    private int spaceBefore;
-    private int spaceAfter;
-    private ColorType backgroundColor;
-    private LengthRangeProperty ipd;
-    private int _height;
-
-    private boolean bAutoLayout = false;
-    private int contentWidth = 0; // Sum of column widths
-    /** Optimum inline-progression-dimension */
-    private int optIPD;
-    /** Minimum inline-progression-dimension */
-    private int minIPD;
-    /** Maximum inline-progression-dimension */
-    private int maxIPD;
 
     /**
      * @param parent FONode that is the parent of this object
@@ -156,29 +137,6 @@ public class Table extends FObj {
         getFOEventHandler().endTable(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.breakBefore = getPropEnum(PR_BREAK_BEFORE);
-        this.breakAfter = getPropEnum(PR_BREAK_AFTER);
-        this.spaceBefore = getPropLength(PR_SPACE_BEFORE | CP_OPTIMUM);
-        this.spaceAfter = getPropLength(PR_SPACE_AFTER | CP_OPTIMUM);
-        this.backgroundColor =
-          this.propertyList.get(PR_BACKGROUND_COLOR).getColorType();
-        this.ipd = this.propertyList.get(
-                     PR_INLINE_PROGRESSION_DIMENSION).getLengthRange();
-        this._height = getPropLength(PR_HEIGHT);
-        this.bAutoLayout = (getPropEnum(PR_TABLE_LAYOUT) == TableLayout.AUTO);
-
-        this.omitHeaderAtBreak = getPropEnum(PR_TABLE_OMIT_HEADER_AT_BREAK)
-            == TableOmitHeaderAtBreak.TRUE;
-        this.omitFooterAtBreak = getPropEnum(PR_TABLE_OMIT_FOOTER_AT_BREAK)
-            == TableOmitFooterAtBreak.TRUE;
-        getFOEventHandler().startTable(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#addChildNode(FONode)
      */
index 80cc2e51db35a5cddca99eef961d47236074b749..e3ed61007aa1788317e0b94588b701b71d8d9e26 100644 (file)
@@ -22,11 +22,9 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.fo.FONode;
 import org.apache.fop.fo.FObj;
 import org.apache.fop.fo.PropertyList;
@@ -53,10 +51,6 @@ public class TableBody extends FObj {
     // private ToBeImplementedProperty visibility;
     // End of property values
     
-    private int spaceBefore;
-    private int spaceAfter;
-    private ColorType backgroundColor;
-
     /**
      * @param parent FONode that is the parent of the object
      */
@@ -93,18 +87,6 @@ public class TableBody extends FObj {
         getFOEventHandler().endBody(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.spaceBefore = getPropLength(PR_SPACE_BEFORE | CP_OPTIMUM);
-        this.spaceAfter = getPropLength(PR_SPACE_AFTER | CP_OPTIMUM);
-        this.backgroundColor =
-          this.propertyList.get(PR_BACKGROUND_COLOR).getColorType();
-        getFOEventHandler().startBody(this);
-    }
-
     /**
      * Return the Common Border, Padding, and Background Properties.
      */
index d0917e36b401520034b6c993ad5a5a22f676d080..8ca7ca2edf94c4c3a7ace05d0f61baf704b58e1f 100644 (file)
@@ -22,12 +22,10 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.datatypes.Length;
 import org.apache.fop.datatypes.Numeric;
 import org.apache.fop.fo.FONode;
@@ -73,14 +71,6 @@ public class TableCell extends FObj {
     private Length width;
     // End of property values
 
-    // private int spaceBefore;
-    // private int spaceAfter;
-    private ColorType backgroundColor;
-
-    private int numColumnsSpanned;
-    private int numRowsSpanned;
-    private int iColNumber = -1;    // uninitialized
-
     /** used for FO validation */
     private boolean blockItemFound = false;
 
@@ -90,12 +80,6 @@ public class TableCell extends FObj {
      */
     protected int startOffset;
 
-    /**
-     * Dimension of allocation rectangle in inline-progression-direction,
-     * determined by the width of the column(s) occupied by the cell
-     */
-    protected int _width;
-
     /**
      * Offset of content rectangle, in block-progression-direction,
      * relative to the row.
@@ -117,35 +101,14 @@ public class TableCell extends FObj {
     /** For collapsed border style */
     protected int borderHeight = 0;
 
-    /** Minimum content height of cell. */
-    protected int minCellHeight = 0;
-
-    /** Height of cell */
-    protected int _height = 0;
-
     /** Ypos of cell ??? */
     protected int top;
 
-    /** corresponds to display-align property */
-    protected int verticalAlign;
-
-    /** is this cell relatively aligned? */
-    protected boolean bRelativeAlign = false;
-
-    // boolean setup = false;
-    private boolean bSepBorders = true;
-
     /**
      * Set to true if all content completely laid out.
      */
     private boolean bDone = false;
 
-    /**
-     * Border separation value in the block-progression dimension.
-     * Used in calculating cells height.
-     */
-    private int _borderSeparation = 0;
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -202,48 +165,6 @@ public class TableCell extends FObj {
         getFOEventHandler().endCell(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.iColNumber =
-            propertyList.get(PR_COLUMN_NUMBER).getNumber().intValue();
-        if (iColNumber < 0) {
-            iColNumber = 0;
-        }
-        this.numColumnsSpanned =
-            this.propertyList.get(PR_NUMBER_COLUMNS_SPANNED).getNumber().intValue();
-        if (numColumnsSpanned < 1) {
-            numColumnsSpanned = 1;
-        }
-        this.numRowsSpanned =
-            this.propertyList.get(PR_NUMBER_ROWS_SPANNED).getNumber().intValue();
-        if (numRowsSpanned < 1) {
-            numRowsSpanned = 1;
-        }
-
-        this.backgroundColor =
-            this.propertyList.get(PR_BACKGROUND_COLOR).getColorType();
-
-        bSepBorders = (getPropEnum(PR_BORDER_COLLAPSE) == BorderCollapse.SEPARATE);
-
-        calcBorders(propMgr.getBorderAndPadding());
-
-        // Vertical cell alignment
-        verticalAlign = getPropEnum(PR_DISPLAY_ALIGN);
-        if (verticalAlign == DisplayAlign.AUTO) {
-            // Depends on all cells starting in row
-            bRelativeAlign = true;
-            verticalAlign = getPropEnum(PR_RELATIVE_ALIGN);
-        } else {
-            bRelativeAlign = false;    // Align on a per-cell basis
-        }
-
-        this.minCellHeight = getPropLength(PR_HEIGHT);
-        getFOEventHandler().startCell(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (%block;)+
@@ -268,43 +189,12 @@ public class TableCell extends FObj {
         startOffset = offset;
     }
 
-    /**
-     * Sets the width of the cell. Initially this width is the same as the
-     * width of the column containing this cell, or the sum of the spanned
-     * columns if numColumnsSpanned > 1
-     * @param width the width of the cell (in millipoints ??)
-     */
-    public void setWidth(int width) {
-        this._width = width;
-    }
-
-    /**
-     * @return number of the column containing this cell
-     */
-    public int getColumnNumber() {
-        return iColNumber;
-    }
-
-    /**
-     * @return the number of columns spanned by this cell
-     */
-    public int getNumColumnsSpanned() {
-        return numColumnsSpanned;
-    }
-
-    /**
-     * @return the number of rows spanned by this cell
-     */
-    public int getNumRowsSpanned() {
-        return numRowsSpanned;
-    }
-
     /**
      * Calculate cell border and padding, including offset of content
      * rectangle from the theoretical grid position.
      */
     private void calcBorders(CommonBorderAndPadding bp) {
-        if (this.bSepBorders) {
+        if (this.borderCollapse == BorderCollapse.SEPARATE) {
             /*
              * Easy case.
              * Cell border is the property specified directly on cell.
@@ -323,9 +213,9 @@ public class TableCell extends FObj {
                                + bp.getPaddingEnd(false);
 
             // Offset of content rectangle in the block-progression direction
-            _borderSeparation = getPropLength(PR_BORDER_SEPARATION | 
+            int bSep = getPropLength(PR_BORDER_SEPARATION | 
                 CP_BLOCK_PROGRESSION_DIRECTION);
-            this.beforeOffset = _borderSeparation / 2
+            this.beforeOffset = bSep / 2
                                 + bp.getBorderBeforeWidth(false)
                                 + bp.getPaddingBefore(false);
 
@@ -403,11 +293,10 @@ public class TableCell extends FObj {
     }
 
     /**
-     * @return number of the column containing this cell
-     * TODO 31699
+     * Return the "column-number" property.
      */
-    public int ___getColumnNumber() {
-        return columnNumber.getValue();
+    public int getColumnNumber() {
+        return Math.max(columnNumber.getValue(), 0);
     }
 
     /**
@@ -418,17 +307,17 @@ public class TableCell extends FObj {
     }
 
     /**
-     * @return the number of columns spanned by this cell
+     * Return the "number-columns-spanned" property.
      */
     public int getNumberColumnsSpanned() {
-        return numberColumnsSpanned.getValue();
+        return Math.max(numberColumnsSpanned.getValue(), 1);
     }
 
     /**
-     * @return the number of rows spanned by this cell
+     * Return the "number-rows-spanned" property.
      */
     public int getNumberRowsSpanned() {
-        return numberRowsSpanned.getValue();
+        return Math.max(numberRowsSpanned.getValue(), 1);
     }
 
     /**
index ad242ede02f0c4864cde89ffc67941d98533242c..522225f4321d87f1e161f50c7c6a131a52d7cf95 100644 (file)
 package org.apache.fop.fo.flow;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.datatypes.Length;
 import org.apache.fop.datatypes.Numeric;
 import org.apache.fop.fo.FONode;
@@ -49,11 +47,6 @@ public class TableColumn extends FObj {
     // private ToBeImplementedProperty visibility;
     // End of property values
     
-    private ColorType backgroundColor;
-    private int columnOffset;
-    private int numColumnsRepeated;
-    private int iColumnNumber;
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -100,29 +93,6 @@ public class TableColumn extends FObj {
             invalidChildError(loc, nsURI, localName);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties(Attributes)
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        iColumnNumber = propertyList.get(PR_COLUMN_NUMBER).getNumber().intValue();
-        numColumnsRepeated =
-            propertyList.get(PR_NUMBER_COLUMNS_REPEATED).getNumber().intValue();
-        this.backgroundColor =
-            this.propertyList.get(PR_BACKGROUND_COLOR).getColorType();
-        columnWidth = this.propertyList.get(PR_COLUMN_WIDTH).getLength();
-
-        getFOEventHandler().startColumn(this);
-    }
-
-    /**
-     * @return value for number of columns repeated
-     */
-    public int getNumColumnsRepeated() {
-        return numColumnsRepeated;
-    }
-
     /**
      * Return the Common Border, Padding, and Background Properties.
      */
index e1c5774d3a7911101717c87fe1a5b320524b8ce5..439f1888792461dacc5f4133aeeac936009776d1 100644 (file)
@@ -22,12 +22,10 @@ package org.apache.fop.fo.flow;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
 // FOP
-import org.apache.fop.datatypes.ColorType;
 import org.apache.fop.datatypes.KeepValue;
 import org.apache.fop.datatypes.Length;
 import org.apache.fop.fo.FONode;
@@ -70,14 +68,6 @@ public class TableRow extends FObj {
     
     private boolean setup = false;
 
-    private ColorType backgroundColor;
-
-    private KeepValue _keepWithNext;
-    private KeepValue _keepWithPrevious;
-    private KeepValue _keepTogether;
-
-    private int minHeight = 0;    // force row height
-
     /**
      * @param parent FONode that is the parent of this object
      */
@@ -126,14 +116,6 @@ public class TableRow extends FObj {
         getFOEventHandler().endRow(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        getFOEventHandler().startRow(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: (table-cell+)
@@ -145,30 +127,6 @@ public class TableRow extends FObj {
         }
     }
 
-    /**
-     * @return keepWithPrevious
-     */
-    public KeepValue getKeepWithPrevious() {
-        return _keepWithPrevious;
-    }
-
-    /**
-     * @todo see if should remove, or move code to addProperties()
-     */
-    private void doSetup() {
-        this.breakAfter = getPropEnum(PR_BREAK_AFTER);
-        this.backgroundColor =
-            this.propertyList.get(PR_BACKGROUND_COLOR).getColorType();
-
-        this._keepTogether = getKeepValue(PR_KEEP_TOGETHER | CP_WITHIN_COLUMN);
-        this._keepWithNext = getKeepValue(PR_KEEP_WITH_NEXT | CP_WITHIN_COLUMN);
-        this._keepWithPrevious =
-            getKeepValue(PR_KEEP_WITH_PREVIOUS | CP_WITHIN_COLUMN);
-
-        this.minHeight = getPropLength(PR_HEIGHT);
-        setup = true;
-    }
-
     private KeepValue getKeepValue(int propId) {
         Property p = this.propertyList.get(propId);
         Number n = p.getNumber();
index 33e2b5caaac6f2e748c3c3757835839b89677a27..8d9b4d0e87c5ad0792d83d4510378c602504ca99 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.pagination;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -46,8 +45,6 @@ public class ConditionalPageMasterReference extends FObj {
     
     private RepeatablePageMasterAlternatives repeatablePageMasterAlternatives;
 
-    private String masterName;
-
     /**
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
@@ -81,38 +78,6 @@ public class ConditionalPageMasterReference extends FObj {
        invalidChildError(loc, nsURI, localName);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        if (getProperty(PR_MASTER_REFERENCE) != null) {
-            setMasterName(getProperty(PR_MASTER_REFERENCE).getString());
-        }
-
-        validateParent(parent);
-
-        this.pagePosition = getPropEnum(PR_PAGE_POSITION);
-        this.oddOrEven = getPropEnum(PR_ODD_OR_EVEN);
-        this.blankOrNotBlank = getPropEnum(PR_BLANK_OR_NOT_BLANK);
-    }
-
-    /**
-     * Sets the master name.
-     * @param masterName name for the master
-     */
-    protected void setMasterName(String masterName) {
-        this.masterName = masterName;
-    }
-
-    /**
-     * Returns the "master-name" attribute of this page master reference
-     * @return the master name
-     */
-    public String getMasterName() {
-        return masterName;
-    }
-
     /**
      * Check if the conditions for this reference are met.
      * checks the page number and emptyness to determine if this
@@ -179,7 +144,7 @@ public class ConditionalPageMasterReference extends FObj {
             this.repeatablePageMasterAlternatives =
                 (RepeatablePageMasterAlternatives)parent;
 
-            if (getMasterName() == null) {
+            if (getMasterReference() == null) {
                 getLogger().warn("single-page-master-reference"
                                        + "does not have a master-name and so is being ignored");
             } else {
index 09cafb3572b4a4efff3e9dfe96a0615ad197ddaa..33f42ca90befa33d9b64a4d5d269505c81b774b2 100644 (file)
@@ -23,7 +23,6 @@ import java.util.ArrayList;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -47,11 +46,6 @@ public class Flow extends FObj {
      */
     private ArrayList markerSnapshot;
 
-    /**
-     * Content-width of current column area during layout
-     */
-    private int contentWidth;
-
     /** used for FO validation */
     private boolean blockItemFound = false;
 
@@ -114,22 +108,6 @@ public class Flow extends FObj {
         getFOEventHandler().endFlow(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        // check flow_name property
-        String flowName = getPropString(PR_FLOW_NAME);
-
-        if (flowName == null || flowName.equals("")) {
-            missingPropertyError("flow-name");
-        }
-
-        getFOEventHandler().startFlow(this);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: marker* (%block;)+
@@ -147,21 +125,6 @@ public class Flow extends FObj {
         }
     }
 
-    /**
-     * @param contentWidth content width of this flow, in millipoints (??)
-     */
-    protected void setContentWidth(int contentWidth) {
-        this.contentWidth = contentWidth;
-    }
-
-    /**
-     * @return the content width of this flow (really of the region
-     * in which it is flowing), in millipoints (??).
-     */
-    public int getContentWidth() {
-        return this.contentWidth;
-    }
-
     /**
      * @return true (Flow can generate reference areas)
      */
@@ -170,7 +133,7 @@ public class Flow extends FObj {
     }
 
     /**
-     * @return the name of this flow
+     * Return "flow-name" property.
      */
     public String getFlowName() {
         return flowName;
index 7697a834d34c10ce4102be1770467cddcd1fc623..40073f76146d20172ccc6b3642c634b6eb3dc449 100644 (file)
@@ -23,7 +23,6 @@ import java.util.Iterator;
 import java.util.Map;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -71,6 +70,8 @@ public class LayoutMasterSet extends FObj {
             throw new SAXParseException("fo:layout-master-set must be child of fo:root, not "
                                    + parent.getName(), locator);
         }
+        simplePageMasters = new java.util.HashMap();
+        pageSequenceMasters = new java.util.HashMap();
     }
 
     /**
@@ -83,24 +84,6 @@ public class LayoutMasterSet extends FObj {
         checkRegionNames();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        if (parent.getName().equals("fo:root")) {
-            Root root = (Root)parent;
-            root.setLayoutMasterSet(this);
-        } else {
-            throw new SAXParseException("fo:layout-master-set must be child of fo:root, not "
-                                   + parent.getName(), locator);
-        }
-
-        this.simplePageMasters = new java.util.HashMap();
-        this.pageSequenceMasters = new java.util.HashMap();
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
         XSL/FOP: (simple-page-master|page-sequence-master)+
index 5205b0e7774cac57af32709a6a6b947cfa12666e..939b8eb83240b8cef6d1d6d832b7ff5f4e882e0e 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.pagination;
 import java.util.HashMap;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -204,58 +203,6 @@ public class PageSequence extends FObj {
         getFOEventHandler().endPageSequence(this);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        this.root = (Root) parent;
-        layoutMasterSet = root.getLayoutMasterSet();
-        flowMap = new HashMap();
-
-        // we are now on the first page of the page sequence
-        thisIsFirstPage = true;
-        Property ipnValue = getProperty(PR_INITIAL_PAGE_NUMBER);
-
-        if (ipnValue.getEnum() != 0) {
-            // auto | auto-odd | auto-even.
-            pageNumberType = ipnValue.getEnum();
-        } else {
-            pageNumberType = EXPLICIT;
-            int pageStart = ipnValue.getNumber().intValue();
-            this.explicitFirstNumber = (pageStart > 0) ? pageStart : 1;
-        }
-
-        String masterName = getPropString(PR_MASTER_REFERENCE);
-        this.simplePageMaster =
-                this.layoutMasterSet.getSimplePageMaster(masterName);
-        if (this.simplePageMaster == null) {
-            this.pageSequenceMaster =
-                    this.layoutMasterSet.getPageSequenceMaster(masterName);
-            if (this.pageSequenceMaster == null) {
-                throw new SAXParseException("master-reference '" + masterName
-                    + "' for fo:page-sequence matches no"
-                    + " simple-page-master or page-sequence-master", locator);
-            }
-        }
-
-        // get the 'format' properties
-        this.pageNumberGenerator =
-            new PageNumberGenerator(getPropString(PR_FORMAT),
-                this.propertyList.get(PR_GROUPING_SEPARATOR).getCharacter(),
-                this.propertyList.get(PR_GROUPING_SIZE).getNumber().intValue(),
-                getPropEnum(PR_LETTER_VALUE));
-
-        this.forcePageCount = getPropEnum(PR_FORCE_PAGE_COUNT);
-
-        // this.propertyList.get("country");
-        // this.propertyList.get("language");
-
-        //call startStructuredPageSequence to ensure, that startPageSequence is called
-        //before startFlow.
-        startStructuredPageSequence();
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
         XSL Content Model: (title?,static-content*,flow)
index fcb2e0d97c65f68af15e54cdf9ed3ebd36e7cdf2..a6713ad2fce7bc8c764bb229f78a4700eb13509c 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.pagination;
 import java.util.List;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -98,28 +97,6 @@ public class PageSequenceMaster extends FObj {
         }
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        subSequenceSpecifiers = new java.util.ArrayList();
-        if (parent.getName().equals("fo:layout-master-set")) {
-            this.layoutMasterSet = (LayoutMasterSet)parent;
-            masterName = getPropString(PR_MASTER_NAME);
-            if (masterName == null) {
-                getLogger().warn("page-sequence-master does not have "
-                                       + "a master-name and so is being ignored");
-            } else {
-                this.layoutMasterSet.addPageSequenceMaster(masterName, this);
-            }
-        } else {
-            throw new SAXParseException("fo:page-sequence-master must be child "
-                                   + "of fo:layout-master-set, not "
-                                   + parent.getName(), locator);
-        }
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL/FOP: (single-page-master-reference|repeatable-page-master-reference|
index a165dcf117687ca4fa3e3a1d6794e5dc6e52f3b7..d6896144158f80a9744b625c8ca86c8d8d0e3f74 100644 (file)
@@ -21,7 +21,6 @@ package org.apache.fop.fo.pagination;
 import java.awt.Rectangle;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -49,9 +48,6 @@ public abstract class Region extends FObj {
     
     private SimplePageMaster layoutMaster;
 
-    /** Holds the writing mode */
-    protected int wm;
-
     /**
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
@@ -89,32 +85,6 @@ public abstract class Region extends FObj {
         }
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        // regions may have name, or default
-        if (null == this.propertyList.get(PR_REGION_NAME)) {
-            setRegionName(getDefaultRegionName());
-        } else if (getPropString(PR_REGION_NAME).equals("")) {
-            setRegionName(getDefaultRegionName());
-        } else {
-            setRegionName(getPropString(PR_REGION_NAME));
-            // check that name is OK. Not very pretty.
-            if (isReserved(getRegionName())
-                    && !getRegionName().equals(getDefaultRegionName())) {
-                throw new SAXParseException("region-name '" + regionName
-                        + "' for " + this.getName()
-                        + " is not permitted.", locator);
-            }
-        }
-
-        this.wm = getPropEnum(PR_WRITING_MODE);
-        this.overflow = getPropEnum(PR_OVERFLOW);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: empty
index 199c2c0dc62e9c47c296ef8b0249ce301e6efdf5..2b47a24f53d64dabcc808b13639ee0ade25cf420 100644 (file)
@@ -21,10 +21,6 @@ package org.apache.fop.fo.pagination;
 // Java
 import java.awt.Rectangle;
 
-// XML
-import org.xml.sax.Attributes;
-import org.xml.sax.SAXParseException;
-
 // FOP
 import org.apache.fop.fo.FONode;
 import org.apache.fop.datatypes.FODimension;
@@ -34,9 +30,6 @@ import org.apache.fop.datatypes.FODimension;
  */
 public class RegionAfter extends RegionBA {
 
-    private int extent = 0;
-    private boolean precedence = false;
-
     /**
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
@@ -44,36 +37,19 @@ public class RegionAfter extends RegionBA {
         super(parent);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        extent = getPropLength(PR_EXTENT);
-        precedence = (getPropEnum(PR_PRECEDENCE) == Precedence.TRUE);
-    }
-
-    /**
-     * Indicates if this region gets precedence.
-     * @return True if it gets precedence
-     */
-    public boolean getPrecedence() {
-        return precedence;
-    }
-
     /**
      * @see org.apache.fop.fo.pagination.Region#getViewportRectangle(FODimension)
      */
     public Rectangle getViewportRectangle (FODimension reldims) {
         // Depends on extent, precedence ans writing mode
         Rectangle vpRect;
-        if (this.wm == WritingMode.LR_TB || this.wm == WritingMode.RL_TB) {
-            vpRect = new Rectangle(0, reldims.bpd - extent, reldims.ipd, extent);
+        if (this.getWritingMode() == WritingMode.LR_TB || this.getWritingMode() == WritingMode.RL_TB) {
+            vpRect = new Rectangle(0, reldims.bpd - getExtent().getValue(), reldims.ipd, getExtent().getValue());
         } else {
-            vpRect = new Rectangle(0, reldims.bpd - extent, extent, reldims.ipd);
+            vpRect = new Rectangle(0, reldims.bpd - getExtent().getValue(), getExtent().getValue(), reldims.ipd);
         }
-        if (precedence == false) {
-            adjustIPD(vpRect, this.wm);
+        if (getPrecedence() == FALSE) {
+            adjustIPD(vpRect, this.getWritingMode());
         }
         return vpRect;
     }
index 4ad44e1687354d17ecee086780f3aa69d64a3474..34fba9c91238a71d496739d4fa855c2c573a013c 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.pagination;
 import java.awt.Rectangle;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 // FOP
@@ -55,13 +54,6 @@ public abstract class RegionBA extends Region {
         precedence = pList.get(PR_PRECEDENCE).getEnum();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-    }
-
     /**
      * Return the "extent" property.
      */
@@ -71,9 +63,8 @@ public abstract class RegionBA extends Region {
 
     /**
      * Return the "precedence" property.
-     * TODO: 31699
      */
-    public int ___getPrecedence() {
+    public int getPrecedence() {
         return precedence;
     }
 
index 1315c2c048e4cc2087cd550f9dbe90f1be4c4403..c00a40ce0cd092f2d354c4b7540818acf5b9ab66 100644 (file)
@@ -21,10 +21,6 @@ package org.apache.fop.fo.pagination;
 // Java
 import java.awt.Rectangle;
 
-// XML
-import org.xml.sax.Attributes;
-import org.xml.sax.SAXParseException;
-
 // FOP
 import org.apache.fop.datatypes.FODimension;
 import org.apache.fop.fo.FONode;
@@ -33,10 +29,6 @@ import org.apache.fop.fo.FONode;
  * The fo:region-before element.
  */
 public class RegionBefore extends RegionBA {
-
-    private int extent = 0;
-    private boolean precedence = false;
-
     /**
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
@@ -44,23 +36,6 @@ public class RegionBefore extends RegionBA {
         super(parent);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        extent = getPropLength(PR_EXTENT);
-        precedence = (getPropEnum(PR_PRECEDENCE) == Precedence.TRUE);
-    }
-
-    /**
-     * Indicates if this region gets precedence.
-     * @return True if it gets precedence
-     */
-    public boolean getPrecedence() {
-        return precedence;
-    }
-
     /**
      * @see org.apache.fop.fo.pagination.Region#getDefaultRegionName()
      */
@@ -79,13 +54,13 @@ public class RegionBefore extends RegionBA {
         // Before is always 0, start depends on extent
         // ipd depends on precedence, bpd=extent
         Rectangle vpRect;
-        if (this.wm == WritingMode.LR_TB || this.wm == WritingMode.RL_TB) {
-            vpRect = new Rectangle(0, 0, reldims.ipd, extent);
+        if (this.getWritingMode() == WritingMode.LR_TB || this.getWritingMode() == WritingMode.RL_TB) {
+            vpRect = new Rectangle(0, 0, reldims.ipd, getExtent().getValue());
         } else {
-            vpRect = new Rectangle(0, 0, extent, reldims.ipd);
+            vpRect = new Rectangle(0, 0, getExtent().getValue(), reldims.ipd);
         }
-        if (precedence == false) {
-            adjustIPD(vpRect, this.wm);
+        if (getPrecedence() == FALSE) {
+            adjustIPD(vpRect, this.getWritingMode());
         }
         return vpRect;
     }
index bfadeb48b392f57acbdc4266b1e24f20b32e8db3..0940719bfc21f672740dbbba764c7d69d55d6236 100644 (file)
@@ -21,10 +21,6 @@ package org.apache.fop.fo.pagination;
 // Java
 import java.awt.Rectangle;
 
-// XML
-import org.xml.sax.Attributes;
-import org.xml.sax.SAXParseException;
-
 // FOP
 import org.apache.fop.fo.FONode;
 import org.apache.fop.datatypes.FODimension;
@@ -33,9 +29,6 @@ import org.apache.fop.datatypes.FODimension;
  * The fo:region-end element.
  */
 public class RegionEnd extends RegionSE {
-
-    private int extent = 0;
-
     /**
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
@@ -43,30 +36,22 @@ public class RegionEnd extends RegionSE {
         super(parent);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        extent = getPropLength(PR_EXTENT);
-    }
-
     /**
      * @see org.apache.fop.fo.pagination.Region#getViewportRectangle(FODimension)
      */
     public Rectangle getViewportRectangle (FODimension reldims) {
         // Depends on extent, precedence and writing mode
         Rectangle vpRect;
-        if (this.wm == WritingMode.LR_TB || this.wm == WritingMode.RL_TB) {
+        if (this.getWritingMode() == WritingMode.LR_TB || this.getWritingMode() == WritingMode.RL_TB) {
             // Rectangle:  x , y (of top left point), width, height
-            vpRect = new Rectangle(reldims.ipd - extent, 0,
-                    extent, reldims.bpd);
+            vpRect = new Rectangle(reldims.ipd - getExtent().getValue(), 0,
+                    getExtent().getValue(), reldims.bpd);
         } else {
             // Rectangle:  x , y (of top left point), width, height
-            vpRect = new Rectangle(reldims.ipd - extent, 0,
-                    reldims.bpd, extent);
+            vpRect = new Rectangle(reldims.ipd - getExtent().getValue(), 0,
+                    reldims.bpd, getExtent().getValue());
         }
-        adjustIPD(vpRect, this.wm);
+        adjustIPD(vpRect, this.getWritingMode());
         return vpRect;
     }
 
index 4ba8b1e7c2217cb8f931d923e759a975eca6ea98..dc8839c6dd84f0eb41f853329d03998f94151bee 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.pagination;
 import java.awt.Rectangle;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.SAXParseException;
 
 // FOP
@@ -53,13 +52,6 @@ public abstract class RegionSE extends Region {
         extent = pList.get(PR_EXTENT).getLength();
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-    }
-
     /**
      * Return the "extent" property.
      */
@@ -80,12 +72,12 @@ public abstract class RegionSE extends Region {
     protected void adjustIPD(Rectangle vpRefRect, int wm) {
         int offset = 0;
         RegionBefore before = (RegionBefore) getSiblingRegion(FO_REGION_BEFORE);
-        if (before != null && before.getPrecedence()) {
+        if (before != null && before.getPrecedence() == TRUE) {
             offset = before.getPropLength(PR_EXTENT);
             vpRefRect.translate(0, offset);
         }
         RegionAfter after = (RegionAfter) getSiblingRegion(FO_REGION_AFTER);
-        if (after != null && after.getPrecedence()) {
+        if (after != null && after.getPrecedence() == TRUE) {
             offset += after.getPropLength(PR_EXTENT);
         }
         if (offset > 0) {
index f00414934d936575415df76b95378101b39b48c2..ad72e2737c8b0cdf24f6ffcc002116c356a45246 100644 (file)
@@ -21,10 +21,6 @@ package org.apache.fop.fo.pagination;
 // Java
 import java.awt.Rectangle;
 
-// XML
-import org.xml.sax.Attributes;
-import org.xml.sax.SAXParseException;
-
 // FOP
 import org.apache.fop.fo.FONode;
 import org.apache.fop.datatypes.FODimension;
@@ -33,9 +29,6 @@ import org.apache.fop.datatypes.FODimension;
  * The fo:region-start element.
  */
 public class RegionStart extends RegionSE {
-
-    private int extent = 0;
-
     /**
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
@@ -43,14 +36,6 @@ public class RegionStart extends RegionSE {
         super(parent);
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        extent = getPropLength(PR_EXTENT);
-    }
-
     /**
      * @see org.apache.fop.fo.pagination.Region#getViewportRectangle(FODimension)
      */
@@ -59,12 +44,12 @@ public class RegionStart extends RegionSE {
         // This is the rectangle relative to the page-reference area in
         // writing-mode relative coordinates
         Rectangle vpRect;
-        if (this.wm == WritingMode.LR_TB || this.wm == WritingMode.RL_TB) {
-            vpRect = new Rectangle(0, 0, extent, reldims.bpd);
+        if (this.getWritingMode() == WritingMode.LR_TB || this.getWritingMode() == WritingMode.RL_TB) {
+            vpRect = new Rectangle(0, 0, getExtent().getValue(), reldims.bpd);
         } else {
-            vpRect = new Rectangle(0, 0, reldims.bpd, extent);
+            vpRect = new Rectangle(0, 0, reldims.bpd, getExtent().getValue());
         }
-        adjustIPD(vpRect, this.wm);
+        adjustIPD(vpRect, this.getWritingMode());
         return vpRect;
     }
 
index c2d60155146084a7d9b7b88af171024bcf17f624..541cf8330015db3f5bdbef217bc88eaff922e4bf 100644 (file)
@@ -22,7 +22,6 @@ package org.apache.fop.fo.pagination;
 import java.util.ArrayList;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -123,36 +122,6 @@ public class RepeatablePageMasterAlternatives extends FObj
         }
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-        conditionalPageMasterRefs = new ArrayList();
-
-        if (parent.getName().equals("fo:page-sequence-master")) {
-            PageSequenceMaster pageSequenceMaster = (PageSequenceMaster)parent;
-            pageSequenceMaster.addSubsequenceSpecifier(this);
-        } else {
-            throw new SAXParseException("fo:repeatable-page-master-alternatives "
-                                   + "must be child of fo:page-sequence-master, not "
-                                   + parent.getName(), locator);
-        }
-
-        Property mr = getProperty(PR_MAXIMUM_REPEATS);
-
-        if (mr.getEnum() == NO_LIMIT) {
-            this._maximumRepeats = INFINITE;
-        } else {
-            this._maximumRepeats = mr.getNumber().intValue();
-            if (this._maximumRepeats < 0) {
-                getLogger().debug("negative maximum-repeats: "
-                                  + this.maximumRepeats);
-                this._maximumRepeats = 0;
-            }
-        }
-    }
-
     /**
      * Get the next matching page master from the conditional
      * page master references.
@@ -173,7 +142,7 @@ public class RepeatablePageMasterAlternatives extends FObj
             ConditionalPageMasterReference cpmr =
                 (ConditionalPageMasterReference)conditionalPageMasterRefs.get(i);
             if (cpmr.isValid(isOddPage, isFirstPage, isBlankPage)) {
-                return cpmr.getMasterName();
+                return cpmr.getMasterReference();
             }
         }
         return null;
index d01422500cf5a89f2d97f57d942010485c1496a1..1b66cbea279f203f88290bb51444c5d25fd3f481 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.pagination;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -45,7 +44,6 @@ public class RepeatablePageMasterReference extends FObj
     private static final int INFINITE = -1;
 
     private PageSequenceMaster pageSequenceMaster;
-    private int _maximumRepeats;
     private int numberConsumed = 0;
 
     /**
@@ -75,35 +73,7 @@ public class RepeatablePageMasterReference extends FObj
             pageSequenceMaster.addSubsequenceSpecifier(this);
         }
     }
-
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        PageSequenceMaster pageSequenceMaster = (PageSequenceMaster) parent;
-
-        if (getPropString(PR_MASTER_REFERENCE) == null) {
-            missingPropertyError("master-reference");
-        } else {
-            pageSequenceMaster.addSubsequenceSpecifier(this);
-        }
-
-        Property mr = getProperty(PR_MAXIMUM_REPEATS);
-
-        if (mr.getEnum() == NO_LIMIT) {
-            this._maximumRepeats = INFINITE;
-        } else {
-            this._maximumRepeats = mr.getNumber().intValue();
-            if (this._maximumRepeats < 0) {
-                getLogger().debug("negative maximum-repeats: "
-                                  + this.maximumRepeats);
-                this._maximumRepeats = 0;
-            }
-        }
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: empty
@@ -119,8 +89,8 @@ public class RepeatablePageMasterReference extends FObj
     public String getNextPageMasterName(boolean isOddPage,
                                         boolean isFirstPage,
                                         boolean isEmptyPage) {
-        if (_maximumRepeats != INFINITE) {
-            if (numberConsumed < _maximumRepeats) {
+        if (getMaximumRepeats() != INFINITE) {
+            if (numberConsumed < getMaximumRepeats()) {
                 numberConsumed++;
             } else {
                 return null;
index 122f0866d63ed663dd9a1893e3441f72a1d5f779..8f39e759cddff0f8bf3bd4476e05b4d96fce44ce 100644 (file)
@@ -24,7 +24,6 @@ import java.util.Iterator;
 import java.util.Map;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -109,24 +108,6 @@ public class SimplePageMaster extends FObj {
         }
     }
 
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        LayoutMasterSet layoutMasterSet = (LayoutMasterSet) parent;
-
-        if (getPropString(PR_MASTER_NAME) == null) {
-            missingPropertyError("master-name");
-        } else {
-            layoutMasterSet.addSimplePageMaster(this);
-        }
-
-        //Well, there are only 5 regions so we can save a bit of memory here
-        regions = new HashMap(5);
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: (region-body,region-before?,region-after?,region-start?,region-end?)
index 2abcbb5650d5650de1e373a1e02f5b91e11dee04..faa61c0aabceef3d5272c7a3b5cede0b635e5e09 100644 (file)
@@ -19,7 +19,6 @@
 package org.apache.fop.fo.pagination;
 
 // XML
-import org.xml.sax.Attributes;
 import org.xml.sax.Locator;
 import org.xml.sax.SAXParseException;
 
@@ -72,20 +71,6 @@ public class SinglePageMasterReference extends FObj
         }
     }
     
-    /**
-     * @see org.apache.fop.fo.FObj#addProperties
-     */
-    protected void addProperties(Attributes attlist) throws SAXParseException {
-        super.addProperties(attlist);
-
-        PageSequenceMaster pageSequenceMaster = (PageSequenceMaster) parent;
-        if (getPropString(PR_MASTER_REFERENCE) == null) {
-            missingPropertyError("master-reference");
-        } else {
-            pageSequenceMaster.addSubsequenceSpecifier(this);
-        }
-    }
-
     /**
      * @see org.apache.fop.fo.FONode#validateChildNode(Locator, String, String)
      * XSL Content Model: empty