Bug 27794 - [PATCH] Merge LayoutProcessor and LayoutManager interfaces
Summary: [PATCH] Merge LayoutProcessor and LayoutManager interfaces
Status: CLOSED FIXED
Alias: None
Product: Fop - Now in Jira
Classification: Unclassified
Component: page-master/layout (show other bugs)
Version: trunk
Hardware: Other other
: P3 normal
Target Milestone: ---
Assignee: fop-dev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-19 04:01 UTC by Glen Mazza
Modified: 2012-04-01 06:45 UTC (History)
0 users



Attachments
patch (also need to drop LayoutProcessor.java) (52.73 KB, text/plain)
2004-03-19 04:02 UTC, Glen Mazza
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Glen Mazza 2004-03-19 04:01:07 UTC
Team, 

The following patch merges the LayoutProcessor interface back into the 
LayoutManager interface, which returns the meaning of the latter into being the 
minimal interface that a LayoutManager must implement in the LayoutManager 
layout strategy.  (Similar to the Renderer interface in the render packages.)  
This change is primarily to simplify the Layout code.

Keiron separated the two last year in order for their to be more freedom in 
layout strategies, but this is before Victor already gave us the LayoutStrategy 
interfaces, which does much the same thing (except doesn't limit you to 
LayoutManager classes at all.)  Keeping separate interfaces isn't buying us 
anything at this time--rather, we can free up the LayoutManager design by 
reducing the number of methods within it.  (So far, though, that hasn't been a 
concern.)

Peter, would implementing this change have any affect on your Alt-Design and 
its possible integration in HEAD?  I don't think it will, and it will make 
things easier in HEAD.

I'll wait three days, and if no objections, apply this on Sunday (Wash DC time).

Thanks,
Glen
Comment 1 Glen Mazza 2004-03-19 04:02:30 UTC
Created attachment 10864 [details]
patch (also need to drop LayoutProcessor.java)
Comment 2 Glen Mazza 2004-03-21 12:01:41 UTC
Applied.
Comment 3 Glenn Adams 2012-04-01 06:45:26 UTC
batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed