Bug in DocumentImpl.cloneNode?

[Ramkumar Natarajan <nr...@cs.purdue.edu>]

Hi,

This seemed to have lost its way the first time I posted it.

I am not sure if this is a bug or intentional, but the following code
fails:
//.... Parse some xml document using myParser
Document xmlDoc = myParser.getDocument();
xmlDoc = (Document) xmlDoc.cloneNode(true); // Create a deep clone
//.... Try using xmlDoc's nodes

The above code throws a null pointer exception when an attempt is made to
access the contents (Nodes) of xmlDoc after a "supposedly" deep clone.

On looking at the code for DocumentImpl.cloneNode, I found that I could
fix the problem by adding the following lines after line 248 to
DocumentImpl.java (in function cloneNode):
if (syncChildren()) {
        synchronizeChildren();
}

I think this should be the correct behaviour, but as I have no idea about
the DOM spec., I am not sure if it is.  Strangely enough, the first piece
of code works perfectly fine with xerces-j version 1.0.3.  It started
failing after I started using 1.1.1 and still fails in 1.1.2.  It would be
interesting to look at the code for DocumentImpl in 1.0.3, but I am just
too lazy.

A zillion apologies if this is just my stupidity and not an actual "bug"
:-)

cheers,
nrk.