You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by el...@apache.org on 2008/10/31 14:50:36 UTC
svn commit: r709412 -
/mina/trunk/core/src/main/java/org/apache/mina/handler/demux/DemuxingIoHandler.java
Author: elecharny
Date: Fri Oct 31 06:50:36 2008
New Revision: 709412
URL: http://svn.apache.org/viewvc?rev=709412&view=rev
Log:
o Removed the 'final' keyword for the exceptionCaught methods
o Added a message in the javadoc to warn the user who might forget to call the messageHandler when overloading those methods.
Modified:
mina/trunk/core/src/main/java/org/apache/mina/handler/demux/DemuxingIoHandler.java
Modified: mina/trunk/core/src/main/java/org/apache/mina/handler/demux/DemuxingIoHandler.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/handler/demux/DemuxingIoHandler.java?rev=709412&r1=709411&r2=709412&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/handler/demux/DemuxingIoHandler.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/handler/demux/DemuxingIoHandler.java Fri Oct 31 06:50:36 2008
@@ -256,8 +256,17 @@
}
}
+ /**
+ * Invoked when any exception is thrown by user IoHandler implementation
+ * or by MINA. If cause is an instance of IOException, MINA will close the
+ * connection automatically.
+ *
+ * <b>Warning !</b> If you are to overload this method, be aware that you
+ * _must_ call the messageHandler in your own method, otherwise it won't
+ * be called.
+ */
@Override
- public final void exceptionCaught(IoSession session, Throwable cause) throws Exception {
+ public void exceptionCaught(IoSession session, Throwable cause) throws Exception {
ExceptionHandler<Throwable> handler = findExceptionHandler(cause.getClass());
if (handler != null) {
handler.exceptionCaught(session, cause);