You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2007/08/17 09:36:59 UTC

svn commit: r566957 - in /mina: branches/1.0/core/src/main/java/org/apache/mina/filter/codec/ branches/1.1/core/src/main/java/org/apache/mina/filter/codec/ trunk/core/src/main/java/org/apache/mina/filter/codec/

Author: trustin
Date: Fri Aug 17 00:36:58 2007
New Revision: 566957

URL: http://svn.apache.org/viewvc?view=rev&rev=566957
Log:
Fixed issue: DIRMINA-421 (ProtocolEncoder.dispose() is called for every encode() call for UDP)
* Removed the finally block in ProtocolCodecFilter.filterWrite()


Modified:
    mina/branches/1.0/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
    mina/branches/1.1/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
    mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java

Modified: mina/branches/1.0/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
URL: http://svn.apache.org/viewvc/mina/branches/1.0/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java?view=diff&rev=566957&r1=566956&r2=566957
==============================================================================
--- mina/branches/1.0/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java (original)
+++ mina/branches/1.0/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java Fri Aug 17 00:36:58 2007
@@ -209,11 +209,6 @@
                 pee = new ProtocolEncoderException(t);
             }
             throw pee;
-        } finally {
-            // Dispose the encoder if this session is connectionless.
-            if (session.getTransportType().isConnectionless()) {
-                disposeEncoder(session);
-            }
         }
     }
 

Modified: mina/branches/1.1/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
URL: http://svn.apache.org/viewvc/mina/branches/1.1/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java?view=diff&rev=566957&r1=566956&r2=566957
==============================================================================
--- mina/branches/1.1/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java (original)
+++ mina/branches/1.1/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java Fri Aug 17 00:36:58 2007
@@ -219,11 +219,6 @@
                 pee = new ProtocolEncoderException(t);
             }
             throw pee;
-        } finally {
-            // Dispose the encoder if this session is connectionless.
-            if (session.getTransportType().isConnectionless()) {
-                disposeEncoder(session);
-            }
         }
     }
 

Modified: mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java?view=diff&rev=566957&r1=566956&r2=566957
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java Fri Aug 17 00:36:58 2007
@@ -216,11 +216,6 @@
                 pee = new ProtocolEncoderException(t);
             }
             throw pee;
-        } finally {
-            // Dispose the encoder if this session is connectionless.
-            if (session.getService().getMetadata().isConnectionless()) {
-                disposeEncoder(session);
-            }
         }
     }