Re: cvs commit: xml-fop/src/java/org/apache/fop/layoutmgr AbstractLayoutManager.java

[Glen Mazza <gr...@yahoo.com>]

--- spepping@apache.org wrote:

> The property
>   lists are not cloned. 

For future clarity, it may be good to add this
"limitation" of FONode.clone() (and other ones you're
aware of) to its javadoc comment.

>   +    /**
>   +     * Perform a shallow cloning operation,
>   +     * set its parent, and optionally clean the
> list of child nodes
>   +     * @param parent the intended parent of the
> clone
>   +     * @param removeChildren if true, clean the
> list of child nodes
>   +     * @return the cloned FO node
>   +     */
>   +    public FONode clone(FONode parent, boolean
> removeChildren)
>   +        throws FOPException {

Glen