Jeremias Maerki wrote:
> I didn't have time to do any fancy stuff. Feel free to improve.
Sorry Jeremias, but I don’t buy that. I have many of my own things to
take care of without spending time finishing the work of others. Adding
a TODO in comments takes very little time and may be quite helpful to
newcomers discovering the code. Eclipse tells you when a variable is not
used, so unless you have a good reason to keep it that I may have
missed, it makes good sense to just remove it.
At some point we will have to stop just making things work, and start
cleaning and refactoring. Otherwise we will end up having no other
choice than rewriting the entire codebase from scratch once again.
> On 10.03.2008 13:17:53 Vincent Hennebert wrote:
>>> Author: jeremias
>>> Date: Mon Mar 10 03:06:37 2008
>>> New Revision: 635508
>>> URL: http://svn.apache.org/viewvc?rev=635508&view=rev
>>> Fixed NPE in BlockContainerLayoutManager when used as a child of an
>>> inline-level FO. Split IP and BP stack limits in LayoutContext (there's now
>>> a certain amount of redundancy with "refIPD" in LayoutContext which I
>>> didn't resolve).
>> Wouldn’t it make sense to re-use refIPD then? Or otherwise, add a TODO
>> to the set/getRefIPD methods and possibly deprecate them? So that at
>> least we know what should be used or not.
>>> Areas are now generated for block-level FOs when used as children of
>>> inline-level FOs.
>>> ClassCastException in ListLayoutManager.mustKeepTogether() fixed (occured
>>> if used as child of an inline-level FO).
>>> Mon Mar 10 03:06:37 2008
>>> @@ -26,6 +26,7 @@
>>> import org.apache.commons.logging.Log;
>>> import org.apache.commons.logging.LogFactory;
>>> import org.apache.fop.area.Area;
>>> import org.apache.fop.area.LineArea;
>>> import org.apache.fop.area.Trait;
>>> @@ -583,7 +584,7 @@
>>> // Set up constraints for inline level managers
>>> // IPD remaining in line
>>> - MinOptMax availIPD = context.getStackLimit();
>>> + MinOptMax availIPD = context.getStackLimitIP();
>> This variable is used nowhere. Why not just remove it?
Vincent Hennebert Anyware Technologies
Apache FOP Committer FOP Development/Consulting