Hi Andreas,
The member variable fobj was assigned via the super(node) constructor call.
Since a correct assigment is made in a super constructor call to
AbstractBaseLayoutManager, a better fix (if you want to keep the final
status of the variable) would be to simply remove the second ssignment in
TableCellLayoutManager.
-- snip --
public class TableCellLayoutManager extends BlockStackingLayoutManager
implements BlockLevelLayoutManager {
.....
/**
* Create a new Cell layout manager.
* @param node table-cell FO for which to create the LM
* @param pgu primary grid unit for the cell
*/
public TableCellLayoutManager(TableCell node, PrimaryGridUnit pgu) {
* super(node);*
*fobj = node;*
this.primaryGridUnit = pgu;
}
-- snip --
Adrian.
--
2008/5/3 Andreas Delmelle <[EMAIL PROTECTED]>:
> On May 3, 2008, at 21:07, [EMAIL PROTECTED] wrote:
>
>> Author: acumiskey
>> Date: Sat May 3 12:07:18 2008
>> New Revision: 653106
>>
>> URL: http://svn.apache.org/viewvc?rev=653106&view=rev
>> Log:
>> Fixed the build. Assignment on final variable fobj.
>>
>
> Quid? Where is the variable assigned, apart from in the constructors (which
> is perfectly legal)?
>
> If the build failed on your end, it probably points to a file I forgot to
> include in the commit. :/
> No real strong issue for me, but the build seemed to work fine on my end
> after that change...
>
> I don't really see why one would need to change the fobj for an LM after it
> has been assigned, hence why I decided to make it final.
>
>
> Cheers
>
> Andreas
>
--
Adrian.
