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=40667>. 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=40667 ------- Additional Comments From [EMAIL PROTECTED] 2006-10-04 12:15 ------- Andrejus, Thanks for your patch. I have a few comments. 1. Your patch for src/java/org/apache/fop/layoutmgr/PageBreakingAlgorithm.java reads: /** the logger for the class */ protected static Log classLog = LogFactory.getLog(PageBreakingAlgorithm.class); + /** the logger for the instance */ + private static Log log = classLog; Why don't you remove 'protected static Log classLog'? Re the comment, a static variable is for the class, not for the instance. 2. In the LayoutManagers, there is a protected static Log instance in org.apache.fop.layoutmgr.AbstractBaseLayoutManager, assigned to class LayoutManager: protected static Log log = LogFactory.getLog(LayoutManager.class) This is used by all LayoutManagers. I can imagine that you find this too coarse, and prefer to see a different logger for each LayoutManager, but then it would be better to do so for each LayoutManager. Regards, Simon -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.