DO NOT REPLY [Bug 27794] New: - [PATCH] Merge LayoutProcessor and LayoutManager interfaces

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=27794>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=27794

[PATCH] Merge LayoutProcessor and LayoutManager interfaces

           Summary: [PATCH] Merge LayoutProcessor and LayoutManager
                    interfaces
           Product: Fop
           Version: 1.0dev
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: page-master/layout
        AssignedTo: fop-dev@xml.apache.org
        ReportedBy: glenmazza@yahoo.com


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