gmazza      2005/02/10 09:08:24

  Modified:    src/java/org/apache/fop/area RenderPagesModel.java
                        StorePagesModel.java
               src/java/org/apache/fop/layoutmgr
                        PageSequenceLayoutManager.java
  Log:
  Some simplification to StorePagesModel, removal of not very informative 
comment in PSLM.
  
  Revision  Changes    Path
  1.13      +3 -2      
xml-fop/src/java/org/apache/fop/area/RenderPagesModel.java
  
  Index: RenderPagesModel.java
  ===================================================================
  RCS file: 
/home/cvs/xml-fop/src/java/org/apache/fop/area/RenderPagesModel.java,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -r1.12 -r1.13
  --- RenderPagesModel.java     6 Feb 2005 20:48:47 -0000       1.12
  +++ RenderPagesModel.java     10 Feb 2005 17:08:24 -0000      1.13
  @@ -67,6 +67,7 @@
       public RenderPagesModel (FOUserAgent userAgent, int renderType, 
           FontInfo fontInfo, OutputStream stream) throws FOPException {
   
  +        super();
           renderer = RendererFactory.createRenderer(userAgent, renderType);
   
           try {
  @@ -220,12 +221,12 @@
           processOffDocumentItems(pendingODI);
           pendingODI.clear();
           processOffDocumentItems(endDocODI);
  -        
  +
           try {
               renderer.stopRenderer();
           } catch (IOException ex) {
               throw new SAXException(ex);
  -        }        
  +        }
       }
   }
   
  
  
  
  1.14      +11 -35    xml-fop/src/java/org/apache/fop/area/StorePagesModel.java
  
  Index: StorePagesModel.java
  ===================================================================
  RCS file: 
/home/cvs/xml-fop/src/java/org/apache/fop/area/StorePagesModel.java,v
  retrieving revision 1.13
  retrieving revision 1.14
  diff -u -r1.13 -r1.14
  --- StorePagesModel.java      6 Dec 2004 05:37:17 -0000       1.13
  +++ StorePagesModel.java      10 Feb 2005 17:08:24 -0000      1.14
  @@ -34,8 +34,8 @@
    * The pages are stored and can be retrieved in any order.
    */
   public class StorePagesModel extends AreaTreeModel {
  -    private List pageSequence = null;
  -    private List currSequence;
  +    private List pageSequenceList = null;
  +    private List currentPageSequencePageList;
       private List offDocumentItems = new java.util.ArrayList();
   
       protected static Log log = LogFactory.getLog(StorePagesModel.class);
  @@ -44,6 +44,7 @@
        * Create a new store pages model
        */
       public StorePagesModel() {
  +        pageSequenceList = new java.util.ArrayList();
       }
   
       /**
  @@ -52,11 +53,8 @@
        * @param title the title of the page sequence.
        */
       public void startPageSequence(LineArea title) {
  -        if (pageSequence == null) {
  -            pageSequence = new java.util.ArrayList();
  -        }
  -        currSequence = new java.util.ArrayList();
  -        pageSequence.add(currSequence);
  +        currentPageSequencePageList = new java.util.ArrayList();
  +        pageSequenceList.add(currentPageSequencePageList);
       }
   
       /**
  @@ -64,7 +62,7 @@
        * @param page the page to add to the current page sequence
        */
       public void addPage(PageViewport page) {
  -        currSequence.add(page);
  +        currentPageSequencePageList.add(page);
       }
   
       /**
  @@ -72,7 +70,7 @@
        * @return the number of page sequences in the document.
        */
       public int getPageSequenceCount() {
  -        return pageSequence.size();
  +        return pageSequenceList.size();
       }
   
       /**
  @@ -81,7 +79,7 @@
        * @return returns the number of pages in a page sequence
        */
       public int getPageCount(int seq) {
  -        List sequence = (List) pageSequence.get(seq - 1);
  +        List sequence = (List) pageSequenceList.get(seq - 1);
           return sequence.size();
       }
   
  @@ -92,7 +90,7 @@
        * @return the PageViewport for the particular page
        */
       public PageViewport getPage(int seq, int count) {
  -        List sequence = (List) pageSequence.get(seq - 1);
  +        List sequence = (List) pageSequenceList.get(seq - 1);
           return (PageViewport) sequence.get(count);
       }
   
  @@ -100,36 +98,14 @@
        * @see 
org.apache.fop.area.AreaTreeModel#handleOffDocumentItem(OffDocumentItem)
        */
       public void handleOffDocumentItem(OffDocumentItem ext) {
  -        int seq, page;
  -        switch(ext.getWhenToProcess()) {
  -            case OffDocumentItem.IMMEDIATELY:
  -                seq = pageSequence == null ? 0 : pageSequence.size();
  -                page = currSequence == null ? 0 : currSequence.size();
  -                break;
  -            case OffDocumentItem.AFTER_PAGE:
  -                break;
  -            case OffDocumentItem.END_OF_DOC:
  -                break;
  -        }
           offDocumentItems.add(ext);
       }
   
       /**
  -     * Get the list of OffDocumentItems that apply at a particular
  -     * position in the document.
  -     * @param seq the page sequence number
  -     * @param count the page count in the sequence
  +     * Get the list of OffDocumentItems for this StorePagesModel.
        * @return the list of OffDocumentItems
        */
  -    public List getOffDocumentItems(int seq, int count) {
  -        return null;
  -    }
  -
  -    /**
  -     * Get the end of document OffDocumentItems for this store pages model.
  -     * @return the list of end OffDocumentItems
  -     */
  -    public List getEndOffDocumentItems() {
  +    public List getOffDocumentItems() {
           return offDocumentItems;
       }
   
  
  
  
  1.30      +1 -2      
xml-fop/src/java/org/apache/fop/layoutmgr/PageSequenceLayoutManager.java
  
  Index: PageSequenceLayoutManager.java
  ===================================================================
  RCS file: 
/home/cvs/xml-fop/src/java/org/apache/fop/layoutmgr/PageSequenceLayoutManager.java,v
  retrieving revision 1.29
  retrieving revision 1.30
  diff -u -r1.29 -r1.30
  --- PageSequenceLayoutManager.java    7 Feb 2005 11:01:36 -0000       1.29
  +++ PageSequenceLayoutManager.java    10 Feb 2005 17:08:24 -0000      1.30
  @@ -202,7 +202,6 @@
           areaTreeModel.startPageSequence(title);
           log.debug("Starting layout");
   
  -        // this should be done another way
           makeNewPage(false, false);
           createBodyMainReferenceArea();
           createSpan(1);
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to