]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Clean up: removed all reset and resetPosition methods, which pre-date the Knuth era...
authorVincent Hennebert <vhennebert@apache.org>
Fri, 14 Dec 2007 11:32:51 +0000 (11:32 +0000)
committerVincent Hennebert <vhennebert@apache.org>
Fri, 14 Dec 2007 11:32:51 +0000 (11:32 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@604171 13f79535-47bb-0310-9956-ffa450edef68

16 files changed:
src/java/org/apache/fop/layoutmgr/AbstractLayoutManager.java
src/java/org/apache/fop/layoutmgr/BlockContainerLayoutManager.java
src/java/org/apache/fop/layoutmgr/BlockLayoutManager.java
src/java/org/apache/fop/layoutmgr/FlowLayoutManager.java
src/java/org/apache/fop/layoutmgr/LayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/ContentLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/InlineStackingLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/LineLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/TextLayoutManager.java
src/java/org/apache/fop/layoutmgr/list/ListBlockLayoutManager.java
src/java/org/apache/fop/layoutmgr/list/ListItemContentLayoutManager.java
src/java/org/apache/fop/layoutmgr/list/ListItemLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/TableAndCaptionLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/TableCaptionLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/TableCellLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/TableLayoutManager.java

index 21af512fccbb32894ad1cad80a1bd14abade5ed5..f0365d0e472c602c322087c6964d1494ad918f5a 100644 (file)
@@ -129,45 +129,6 @@ public abstract class AbstractLayoutManager extends AbstractBaseLayoutManager
         return childLMiter.hasNext();
     }
 
-    /**
-     * Reset the layoutmanager "iterator" so that it will start
-     * with the passed Position's generating LM
-     * on the next call to getChildLM.
-     * @param pos a Position returned by a child layout manager
-     * representing a potential break decision.
-     * If pos is null, then back up to the first child LM.
-     */
-    protected void reset(org.apache.fop.layoutmgr.Position pos) {
-        //if (lm == null) return;
-        LayoutManager lm = (pos != null) ? pos.getLM() : null;
-        if (curChildLM != lm) {
-            // ASSERT curChildLM == (LayoutManager)childLMiter.previous()
-            if (childLMiter.hasPrevious() && curChildLM
-                    != (LayoutManager) childLMiter.previous()) {
-                //log.error("LMiter problem!");
-            }
-            while (curChildLM != lm && childLMiter.hasPrevious()) {
-                curChildLM.resetPosition(null);
-                curChildLM = (LayoutManager) childLMiter.previous();
-            }
-            // Otherwise next returns same object
-            childLMiter.next();
-        }
-        if (curChildLM != null) {
-            curChildLM.resetPosition(pos);
-        }
-        if (isFinished()) {
-            setFinished(false);
-        }
-    }
-
-    /** {@inheritDoc} */
-    public void resetPosition(Position resetPos) {
-        //  if (resetPos == null) {
-        //      reset(null);
-        //  }
-    }
-
     /**
      * Tell whether this LayoutManager has handled all of its content.
      * @return True if there are no more break possibilities,
index 9c3c8ce593ddda106406740672008dfb5a730ccb..03aa380d298292bbce295a3777b78b562a249a67 100644 (file)
@@ -938,15 +938,6 @@ public class BlockContainerLayoutManager extends BlockStackingLayoutManager
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
-
     /** 
      * Force current area to be added to parent area.
      * {@inheritDoc}
index 62e41ac16900dcef2bec92a316e5d0303dd99bca..bb39def8d668a83e1c3ce57456070bfade02077d 100644 (file)
@@ -476,19 +476,6 @@ public class BlockLayoutManager extends BlockStackingLayoutManager
         }
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-            childBreaks.clear();
-        } else {
-            //reset(resetPos);
-            LayoutManager lm = resetPos.getLM();
-        }
-    }
-
     /**
      * convenience method that returns the Block node
      * @return the block node
index f5d912111bdfe14ca086eed5600a127d4c5b61dd..a70dd0883bb3a62427a717342e158c40095a2d29 100644 (file)
@@ -342,14 +342,6 @@ public class FlowLayoutManager extends BlockStackingLayoutManager
         return parentArea;
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
     /**
      * Returns the IPD of the content area
      * @return the IPD of the content area
index 39b9706821cfb426e6a5516f60c98127c95f814e..0b17c9a27bb346c61926c895d500e02800252448 100644 (file)
@@ -58,13 +58,6 @@ public interface LayoutManager extends PercentBaseContext {
      */
     PageSequenceLayoutManager getPSLM();
 
-    /**
-     * Reset to the position.
-     *
-     * @param position the Position to reset to
-     */
-    void resetPosition(Position position);
-
     /**
      * Return a value indicating whether this LayoutManager has laid out
      * all its content (or generated BreakPossibilities for all content.)
index 22a01f9fb1c8e343c1f6062eb3320016d92f6688..03e9b382a8f383af015c47a86f82287e177b878d 100644 (file)
@@ -212,13 +212,6 @@ public class ContentLayoutManager extends AbstractBaseLayoutManager
         //to be done
     }
 
-    /**
-     * {@inheritDoc}
-     */
-    public void resetPosition(Position position) {
-        //to be done
-    }
-
     /**
      * {@inheritDoc}
      */
index 7da7cc49f8fe55c40e89e0a869de767d459f4340..4d0872a6d36b0765b5f282b36feeeaeb3c660559 100644 (file)
@@ -154,41 +154,6 @@ public abstract class InlineStackingLayoutManager extends AbstractLayoutManager
         return null;
     }
 
-    /**
-     * Reset position for returning next BreakPossibility.
-     * @param prevPos a Position returned by this layout manager
-     * representing a potential break decision.
-     */
-    public void resetPosition(Position prevPos) {
-        if (prevPos != null) {
-            // ASSERT (prevPos.getLM() == this)
-            if (prevPos.getLM() != this) {
-                //getLogger().error(
-                //  "InlineStackingLayoutManager.resetPosition: " +
-                //  "LM mismatch!!!");
-            }
-            // Back up the child LM Position
-            Position childPos = prevPos.getPosition();
-            reset(childPos);
-            /*
-            if (prevBP != null
-                    && prevBP.getLayoutManager() != childPos.getLM()) {
-                childLC = null;
-            }
-            prevBP = new BreakPoss(childPos);
-            */
-        } else {
-            // Backup to start of first child layout manager
-            //prevBP = null;
-            // super.resetPosition(prevPos);
-            reset(prevPos);
-            // If any areas created, we are restarting!
-            bAreaCreated = false;
-        }
-        // Do we need to reset some context like pending or prevContent?
-        // What about prevBP?
-    }
-
     /**
      * TODO: Explain this method
      * @param lm ???
index e722162d8ae1ba65013e0b55a74d8589de785b2b..6df7ac00c8c3f2fcde3c34cf1311f18cef260327 100644 (file)
@@ -1555,33 +1555,6 @@ public class LineLayoutManager extends InlineStackingLayoutManager
         }
     }
 
-    /**
-     * Reset the positions to the given position.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            setFinished(false);
-            iReturnedLBP = 0;
-        } else {
-            if (isFinished()) {
-                // if isFinished is true, iReturned LBP == breakpoints.size()
-                // and breakpoints.get(iReturnedLBP) would generate
-                // an IndexOutOfBoundException
-                setFinished(false);
-                iReturnedLBP--;
-            }
-            // It is not clear that the member lineLayouts has the correct value;
-            // because the method is not called, this cannot be checked.
-            while ((LineBreakPosition) lineLayouts.getChosenPosition(iReturnedLBP)
-                   != (LineBreakPosition) resetPos) {
-                iReturnedLBP--;
-            }
-            iReturnedLBP++;
-        }
-    }
-
     /**
      * Add the areas with the break points.
      *
index 63f11147c5052ba33743584a8a6e7f7f7f56aa97..40e1c087a95a28066f3638fc6acea9136cacd4e2 100644 (file)
@@ -224,36 +224,6 @@ public class TextLayoutManager extends LeafNodeLayoutManager {
 
     }
 
-    /**
-     * Reset position for returning next BreakPossibility.
-     *
-     * @param prevPos the position to reset to
-     */
-    public void resetPosition(Position prevPos) {
-        if (prevPos != null) {
-            // ASSERT (prevPos.getLM() == this)
-            if (prevPos.getLM() != this) {
-                log.error("TextLayoutManager.resetPosition: "
-                          + "LM mismatch!!!");
-            }
-            LeafPosition tbp = (LeafPosition) prevPos;
-            AreaInfo ai = (AreaInfo) vecAreaInfo.get(tbp.getLeafPos());
-            if (ai.iBreakIndex != iNextStart) {
-                iNextStart = ai.iBreakIndex;
-                vecAreaInfo.ensureCapacity(tbp.getLeafPos() + 1);
-                // TODO: reset or recalculate total IPD = sum of all word IPD
-                // up to the break position
-                ipdTotal = ai.ipdArea;
-                setFinished(false);
-            }
-        } else {
-            // Reset to beginning!
-            vecAreaInfo.clear();
-            iNextStart = 0;
-            setFinished(false);
-        }
-    }
-
     // TODO: see if we can use normal getNextBreakPoss for this with
     // extra hyphenation information in LayoutContext
     private boolean getHyphenIPD(HyphContext hc, MinOptMax hyphIPD) {
index acd9bf718b7862dcef96ba4844ce55331b2ce5fd..470cbbe9c018675ebaa551b9da23ea6fbaa663f3 100644 (file)
@@ -277,19 +277,6 @@ public class ListBlockLayoutManager extends BlockStackingLayoutManager
         }
     }
 
-    /**
-     * Reset the position of this layout manager.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        } else {
-            //TODO Something to put here?
-        }
-    }
-    
     /** {@inheritDoc} */
     public boolean mustKeepTogether() {
         //TODO Keeps will have to be more sophisticated sooner or later
index e1171d6fd112282e49737f02e85e1eaf4bca656a..853b1a128a4ef3cffb4689282bdd5dd9ed464887 100644 (file)
@@ -220,20 +220,6 @@ public class ListItemContentLayoutManager extends BlockStackingLayoutManager {
         }
     }
 
-    /**
-     * Reset the position of the layout.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        } else {
-            setFinished(false);
-            //reset(resetPos);
-        }
-    }
-    
     /** {@inheritDoc} */
     public boolean mustKeepTogether() {
         //TODO Keeps will have to be more sophisticated sooner or later
index 7b7cb52b05f2e217d6dd5548eaee50c08c630765..dc28e98e2ab9b63201aae3903ae380957ed73dfa 100644 (file)
@@ -623,17 +623,6 @@ public class ListItemLayoutManager extends BlockStackingLayoutManager
         }
     }
 
-    /**
-     * Reset the position of this layout manager.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
-    
     /** {@inheritDoc} */
     public boolean mustKeepTogether() {
         //TODO Keeps will have to be more sophisticated sooner or later
index 55b67053f96b183d2b69cb85ae2c949847f29d4f..6d2e49e967c533782472b280959e8d5e9d809b49 100644 (file)
@@ -194,16 +194,5 @@ public class TableAndCaptionLayoutManager extends BlockStackingLayoutManager {
             curBlockArea.addBlock((Block) childArea);
         }
     }
-
-    /**
-     * Reset the position of this layout manager.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
 }
 
index 58c85e8cd660b122efc9456a3442e3afb43400dc..4c21df937eb79bcd9700e2535106a8aa6ed8dd86 100644 (file)
@@ -193,16 +193,5 @@ public class TableCaptionLayoutManager extends BlockStackingLayoutManager {
                 curBlockArea.addBlock((Block) childArea);
         }
     }
-
-    /**
-     * Reset the layout position.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
 }
 
index 46bdecf63a44d7298b9c19d99a8061d47f4ea167..63810e05764fbcc1dff6531114e47f893570186c 100644 (file)
@@ -504,17 +504,6 @@ public class TableCellLayoutManager extends BlockStackingLayoutManager
         }
     }
 
-    /**
-     * Reset the position of the layout.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
-
     /**
      * {@inheritDoc}
      */
index e3377369504ff1bb971e4c994fef1a13f4b6b0e7..2e366f36d29a44b26ae81c65f6d21bf0b7c499c5 100644 (file)
@@ -404,17 +404,6 @@ public class TableLayoutManager extends BlockStackingLayoutManager
         }
     }
 
-    /**
-     * Reset the position of this layout manager.
-     *
-     * @param resetPos the position to reset to
-     */
-    public void resetPosition(Position resetPos) {
-        if (resetPos == null) {
-            reset(null);
-        }
-    }
-
     /** {@inheritDoc} */
     public int negotiateBPDAdjustment(int adj, KnuthElement lastElement) {
         // TODO Auto-generated method stub