https://issues.apache.org/bugzilla/show_bug.cgi?id=37579





--- Comment #19 from Andreas L. Delmelle <[EMAIL PROTECTED]>  2008-05-02 
12:48:50 PST ---
(In reply to comment #18)
> Just ran the layoutengine test-suite after having applied the patch, and it
> seems this causes quite a few tests to break. One of the reasons is an
> IndexOutOfBoundsException at TableStepper line#237...
> 

Found the cause of this: the error was that the additional code in TableStepper
assumed the list of activeCells to be always equal to the number of columns.
Changing the code-fragment to be based on activeCells.size() instead of
columnCount did the trick.

Also, but that's maybe more of a personal preference, IMO the
ElementListUtils.collectFootnodeBodyLMs() implementations should better be
turned around for reasons of code-clarity.
Right now, the implementation dealing with a single List creates one-element
arrays and delegates to the implementation for an array of List. Somehow, I
like the opposite where a call to the latter implementation expands into
multiple calls to the 'simple' implementation.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to