Some thoughts and questions regarding this change. Interface XMLElementListener now has a close() method. So far, so good.
But why do notable implementations like MinaStanzaListener and WebSocketBackedSessionContext record the 'closed' state, but beyond that do not honour it? I suggest they should simply stop whatever they do. 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.
Bernd On 19.03.12 04:09, [email protected] wrote:
Author: foodmike Date: Mon Mar 19 03:09:21 2012 New Revision: 1302264 URL: http://svn.apache.org/viewvc?rev=1302264&view=rev Log: Fix to address VYSPER-18. Modifications to XML parsing/handling in order to recognize and act on stream close events. This fix adds a closed flag to the XMLElementListener that allows the XMPPDecoder to determine when a stream is closed.
