Saxon

Legacy ID: #3168832 Legacy Poster: Michael Kay (mhkay)

Thanks, I think I probably found and fixed this same bug last week as a result of running the new XQuery test suite. Did you find a patch that worked? Michael Kay

Legacy ID: #3168848 Legacy Poster: marcvc (marcvc)

at least for the simple scenarios we've ran up to now, simply decreasing the currentDepth in endDocument seems to be sufficient. But if you have a more robust fix, I'm of course looking forward to it. Thanks, Marc

Legacy ID: #3171469 Legacy Poster: marcvc (marcvc)

Michael, an I forgot to mention, "started" should be set to false during endDocument(). While "ended" needs to be set to false in startDocument(). As I indicated before, we don't know if this is bullet proof and the most clean solution. It just seems to work for the sceanrios we've ran up to now. Thanks, Marc

Legacy ID: #3173390 Legacy Poster: marcvc (marcvc)

Michael, after having a closer look, all this doesn't seem sufficient. All the TimyDocumentImpl nodes added to the forest have nodeNr = 0. Somehow, the nodeNr of the TinyDocumentImpl object should be set properly during TinyTree#addDocumentNode()? Thanks, Marc

Legacy ID: #3173743 Legacy Poster: Michael Kay (mhkay)

I'll look into this and report back. Michael Kay