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);