> >>> Another way to go (and I'd prefer that ATM) would be to move close(), >>> isClosed() down to MinaStanzaListener. It is not needed anywhere else. >> >> This makes sense. I'll move it out of XMLElementListener and only implement >> it in MinaStanzaListener. >
Just getting back to looking at this. On further inspection, I think we have to leave close() in XMLElementListener, since XMPPContentHandler holds a reference to XMLElementListener, and XMPPContentHandler is what needs to call close() from within endDocument.
