On Tue, Jul 13, 2004 at 12:16:22AM -0000, [EMAIL PROTECTED] wrote: > gmazza 2004/07/12 17:16:22 > Log: > 1.) Combined the AreaTree and FOTreeHandler into a new AreaTreeHandler > object. FOTreeHandler was primarily acting as an AreaTreeHandler, > and AreaTree had a 1-to-1 relationship with it. Comments most welcome.
This change seems to cross the border between FO tree and area tree. AreaTree had no reference to the fo package (apart from a reference to fo.extensions). AreaTreeHandler extends a class in the fo package and refers to fo/PageSequence. FOInputHandler implements a number of methods of the SAX contenthandler, which is between the fo file and the FO tree. The area package now inherits these methods. AreaTreeHandler throws SAXExceptions. This is a parser based exception type. FOTreeHandler.endPageSequence is the FO tree's activation of a new episode of area tree building. Now the area tree activates itself, based on an event in the FO tree. I believe this change violates the separation between the FO tree and the area tree. I think that separation is a good idea and should be maintained. Regards, Simon -- Simon Pepping home page: http://www.leverkruid.nl