]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Renaming GridUnitPart into the more accurate CellPart. Moreover I was always making...
authorVincent Hennebert <vhennebert@apache.org>
Mon, 17 Dec 2007 18:56:46 +0000 (18:56 +0000)
committerVincent Hennebert <vhennebert@apache.org>
Mon, 17 Dec 2007 18:56:46 +0000 (18:56 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@604965 13f79535-47bb-0310-9956-ffa450edef68

src/java/org/apache/fop/layoutmgr/table/ActiveCell.java
src/java/org/apache/fop/layoutmgr/table/CellPart.java [new file with mode: 0644]
src/java/org/apache/fop/layoutmgr/table/GridUnitPart.java [deleted file]
src/java/org/apache/fop/layoutmgr/table/RowPainter.java
src/java/org/apache/fop/layoutmgr/table/TableContentLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/TableContentPosition.java
src/java/org/apache/fop/layoutmgr/table/TableStepper.java

index 139eacf43c3f88939bfe282fd546074695b53d50..a1957eb27166758814c74bb87c7d75017c928d55 100644 (file)
@@ -266,12 +266,12 @@ class ActiveCell {
     }
 
     /**
-     * Creates and returns a GridUnitPart instance for the content of this cell which
+     * Creates and returns a CellPart instance for the content of this cell which
      * is included in the next step.
      * 
-     * @return a GridUnitPart instance
+     * @return a CellPart instance
      */
-    GridUnitPart createGridUnitPart() {
+    CellPart createCellPart() {
         if (end + 1 == elementList.size()) {
             if (pgu.getFlag(GridUnit.KEEP_WITH_NEXT_PENDING)) {
                 keepWithNextSignal = true;
@@ -284,9 +284,9 @@ class ActiveCell {
                 && elementList.size() == 1
                 && elementList.get(0) instanceof KnuthBoxCellWithBPD) {
             //Special case: Cell with fixed BPD
-            return new GridUnitPart(pgu, 0, pgu.getElements().size() - 1);
+            return new CellPart(pgu, 0, pgu.getElements().size() - 1);
         } else {
-            return new GridUnitPart(pgu, start, end);
+            return new CellPart(pgu, start, end);
         }
     }
 
diff --git a/src/java/org/apache/fop/layoutmgr/table/CellPart.java b/src/java/org/apache/fop/layoutmgr/table/CellPart.java
new file mode 100644 (file)
index 0000000..bf5ad64
--- /dev/null
@@ -0,0 +1,73 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.layoutmgr.table;
+
+import org.apache.fop.fo.flow.table.GridUnit;
+import org.apache.fop.fo.flow.table.PrimaryGridUnit;
+
+/**
+ * Represents a non-dividable part of a grid unit. Used by the table stepper.
+ */
+class CellPart {
+
+    /** Primary grid unit */
+    protected PrimaryGridUnit pgu;
+    /** Index of the starting element of this part */
+    protected int start;
+    /** Index of the ending element of this part */
+    protected int end;
+
+    /**
+     * Creates a new CellPart.
+     * @param pgu Primary grid unit
+     * @param start starting element
+     * @param end ending element
+     */
+    protected CellPart(PrimaryGridUnit pgu, int start, int end) {
+        this.pgu = pgu;
+        this.start = start;
+        this.end = end;
+    }
+
+    /** @return true if this part is the first part of a cell */
+    public boolean isFirstPart() {
+        return (start == 0);
+    }
+
+    /** @return true if this part is the last part of a cell */
+    public boolean isLastPart() {
+        return (end >= 0 && end == pgu.getElements().size() - 1);
+    }
+
+    /** {@inheritDoc} */
+    public String toString() {
+        StringBuffer sb = new StringBuffer("Part: ");
+        sb.append(start).append("-").append(end);
+        sb.append(" [").append(isFirstPart() ? "F" : "-").append(isLastPart() ? "L" : "-");
+        sb.append("] ").append(pgu);
+        return sb.toString();
+    }
+
+    boolean mustKeepWithPrevious() {
+        return pgu.getFlag(GridUnit.KEEP_WITH_PREVIOUS_PENDING)
+                || (pgu.getRow() != null && pgu.getRow().mustKeepWithPrevious());
+    }
+
+}
diff --git a/src/java/org/apache/fop/layoutmgr/table/GridUnitPart.java b/src/java/org/apache/fop/layoutmgr/table/GridUnitPart.java
deleted file mode 100644 (file)
index e0cdfcc..0000000
+++ /dev/null
@@ -1,73 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.layoutmgr.table;
-
-import org.apache.fop.fo.flow.table.GridUnit;
-import org.apache.fop.fo.flow.table.PrimaryGridUnit;
-
-/**
- * Represents a non-dividable part of a grid unit. Used by the table stepper.
- */
-class GridUnitPart {
-
-    /** Primary grid unit */
-    protected PrimaryGridUnit pgu;
-    /** Index of the starting element of this part */
-    protected int start;
-    /** Index of the ending element of this part */
-    protected int end;
-
-    /**
-     * Creates a new GridUnitPart.
-     * @param pgu Primary grid unit
-     * @param start starting element
-     * @param end ending element
-     */
-    protected GridUnitPart(PrimaryGridUnit pgu, int start, int end) {
-        this.pgu = pgu;
-        this.start = start;
-        this.end = end;
-    }
-
-    /** @return true if this part is the first part of a cell */
-    public boolean isFirstPart() {
-        return (start == 0);
-    }
-
-    /** @return true if this part is the last part of a cell */
-    public boolean isLastPart() {
-        return (end >= 0 && end == pgu.getElements().size() - 1);
-    }
-
-    /** {@inheritDoc} */
-    public String toString() {
-        StringBuffer sb = new StringBuffer("Part: ");
-        sb.append(start).append("-").append(end);
-        sb.append(" [").append(isFirstPart() ? "F" : "-").append(isLastPart() ? "L" : "-");
-        sb.append("] ").append(pgu);
-        return sb.toString();
-    }
-
-    boolean mustKeepWithPrevious() {
-        return pgu.getFlag(GridUnit.KEEP_WITH_PREVIOUS_PENDING)
-                || (pgu.getRow() != null && pgu.getRow().mustKeepWithPrevious());
-    }
-
-}
index 73cce29e4da5e91fa2d95f2a283327cbef6a8571..f78d47b52026e0a2052b4de3bd06a276fda75739 100644 (file)
@@ -110,27 +110,27 @@ class RowPainter {
         }
         rowFO = tcpos.row.getTableRow();
         lastRow = tcpos.row;
-        Iterator partIter = tcpos.gridUnitParts.iterator();
+        Iterator partIter = tcpos.cellParts.iterator();
         //Iterate over all grid units in the current step
         while (partIter.hasNext()) {
-            GridUnitPart gup = (GridUnitPart)partIter.next();
+            CellPart cellPart = (CellPart)partIter.next();
             if (log.isDebugEnabled()) {
-                log.debug(">" + gup);
+                log.debug(">" + cellPart);
             }
-            int colIndex = gup.pgu.getStartCol();
-            if (primaryGridUnits[colIndex] != gup.pgu) {
+            int colIndex = cellPart.pgu.getStartCol();
+            if (primaryGridUnits[colIndex] != cellPart.pgu) {
                 if (primaryGridUnits[colIndex] != null) {
                     log.warn("Replacing GU in slot " + colIndex
                             + ". Some content may not be painted.");
                 }
-                primaryGridUnits[colIndex] = gup.pgu;
-                start[colIndex] = gup.start;
-                end[colIndex] = gup.end;
+                primaryGridUnits[colIndex] = cellPart.pgu;
+                start[colIndex] = cellPart.start;
+                end[colIndex] = cellPart.end;
             } else {
-                if (gup.end < end[colIndex]) {
+                if (cellPart.end < end[colIndex]) {
                     throw new IllegalStateException("Internal Error: stepper problem");
                 }
-                end[colIndex] = gup.end;
+                end[colIndex] = cellPart.end;
             }
         }
     }
@@ -175,7 +175,7 @@ class RowPainter {
                 if (forcedFlush || ((end[i] == primaryGridUnits[i].getElements().size() - 1)
                         && (currentGU == null || currentGU.isLastGridUnitRowSpan()))) {
                     //the last line in the "if" above is to avoid a premature end of a
-                    //row-spanned cell because no GridUnitParts are generated after a cell is
+                    //row-spanned cell because no CellParts are generated after a cell is
                     //finished with its content.
                     //See table-cell_number-rows-spanned_bug38397.xml
                     addAreasForCell(primaryGridUnits[i], start[i], end[i], lastRow, partBPD[i],
index fca0557b75163975532a8da498c3eabb8b324eef..57972078eb2b73172ccccbaee4cefc33a2f53ea4 100644 (file)
@@ -387,7 +387,7 @@ public class TableContentLayoutManager implements PercentBaseContext {
             if (pos instanceof TableContentPosition) {
                 TableContentPosition tcpos = (TableContentPosition)pos;
                 lst.add(tcpos);
-                GridUnitPart part = (GridUnitPart)tcpos.gridUnitParts.get(0);
+                CellPart part = (CellPart)tcpos.cellParts.get(0);
                 if (body == null) {
                     body = part.pgu.getBody();
                 }
index 8986ea85fdcc94ba7e55de101def957e13a02439..68f55d88fea68491ba521593500b38ffc853bc2d 100644 (file)
@@ -36,8 +36,8 @@ class TableContentPosition extends Position {
     /** The position is the last of the row group. */
     public static final int LAST_IN_ROWGROUP = 2;
 
-    /** the list of GridUnitParts making up this position */
-    protected List gridUnitParts;
+    /** the list of CellParts making up this position */
+    protected List cellParts;
     /** effective row this position belongs to */
     protected EffRow row;
     /** flags for the position */
@@ -46,13 +46,13 @@ class TableContentPosition extends Position {
     /**
      * Creates a new TableContentPosition.
      * @param lm applicable layout manager
-     * @param gridUnitParts the list of GridUnitPart instances
+     * @param cellParts the list of CellPart instances
      * @param row effective row this position belongs to
      */
-    protected TableContentPosition(LayoutManager lm, List gridUnitParts,
+    protected TableContentPosition(LayoutManager lm, List cellParts,
             EffRow row) {
         super(lm);
-        this.gridUnitParts = gridUnitParts;
+        this.cellParts = cellParts;
         this.row = row;
     }
 
@@ -90,7 +90,7 @@ class TableContentPosition extends Position {
         sb.append(getFlag(FIRST_IN_ROWGROUP) ? "F" : "-");
         sb.append(getFlag(LAST_IN_ROWGROUP) ? "L" : "-").append("]");
         sb.append("(");
-        sb.append(gridUnitParts);
+        sb.append(cellParts);
         sb.append(")");
         return sb.toString();
     }
index 98b07778f7572ba1650da74357bc93232aa49acf..d4ed460b40b141d56d5307d59aa40c18d9265533 100644 (file)
@@ -169,26 +169,27 @@ public class TableStepper {
             boolean forcedBreak = false;
             int breakClass = -1;
             //Put all involved grid units into a list
-            List gridUnitParts = new java.util.ArrayList(maxColumnCount);
+            List cellParts = new java.util.ArrayList(maxColumnCount);
             for (Iterator iter = activeCells.iterator(); iter.hasNext();) {
                 ActiveCell activeCell = (ActiveCell) iter.next();
                 if (activeCell.contributesContent()) {
-                    GridUnitPart gup = activeCell.createGridUnitPart();
-                    gridUnitParts.add(gup);
+                    CellPart part = activeCell.createCellPart();
+                    cellParts.add(part);
                     forcedBreak = activeCell.isLastForcedBreak();
                     if (forcedBreak) {
                         breakClass = activeCell.getLastBreakClass();
                     }
-                    if (returnList.size() == 0 && gup.isFirstPart() && gup.mustKeepWithPrevious()) {
+                    if (returnList.size() == 0 && part.isFirstPart()
+                            && part.mustKeepWithPrevious()) {
                         context.setFlags(LayoutContext.KEEP_WITH_PREVIOUS_PENDING);
                     }
                 }
             }
-            //log.debug(">>> guPARTS: " + gridUnitParts);
+            //log.debug(">>> guPARTS: " + cellParts);
 
             //Create elements for step
             TableContentPosition tcpos = new TableContentPosition(getTableLM(),
-                    gridUnitParts, rowGroup[normalRow]);
+                    cellParts, rowGroup[normalRow]);
             if (returnList.size() == 0) {
                 tcpos.setFlag(TableContentPosition.FIRST_IN_ROWGROUP, true);
             }