Issue Details (XML | Word | Printable)

Key: DIRMINA-109
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Trustin Lee
Reporter: Trustin Lee
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
MINA

CumulativeDecoder doesn't release allocated buffer.

Created: 23/Oct/05 10:15 AM   Updated: 29/Oct/05 12:08 AM
Return to search
Component/s: None
Affects Version/s: 0.8.0
Fix Version/s: 0.9.0

Time Tracking:
Not Specified

Resolution Date: 29/Oct/05 12:08 AM


 Description  « Hide
We had to release the buffer in the dispose() method.

Trustin

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Repository Revision Date User Message
ASF #329220 Fri Oct 28 15:07:17 UTC 2005 trustin Resolved issue: DIRMINA-109 - CumulativeDecoder doesn't release allocated buffer.
* Implemented CumulativeDecoder.dispose() method.
* DemuxingProtocolCodecFactory.ProtocolDecoderImpl now calls super.dispose().
* CumulativeDecoder is now can be a singleton.
Files Changed
MODIFY /directory/network/trunk/src/java/org/apache/mina/filter/codec/DemuxingProtocolCodecFactory.java
MODIFY /directory/network/trunk/src/java/org/apache/mina/filter/codec/ProtocolCodecFactory.java
MODIFY /directory/network/trunk/src/test/org/apache/mina/filter/codec/CumulativeProtocolDecoderTest.java
MODIFY /directory/network/trunk/src/java/org/apache/mina/filter/codec/CumulativeProtocolDecoder.java
MODIFY /directory/network/trunk/src/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
MODIFY /directory/network/trunk/src/examples/org/apache/mina/examples/reverser/ReverseProtocolHandler.java

Trustin Lee added a comment - 28/Oct/05 11:42 PM
Dispose() method exists from 0.9. So we should fix this in 0.9.

Trustin Lee made changes - 28/Oct/05 11:42 PM
Field Original Value New Value
Fix Version/s 0.8.1 [ 12310370 ]
Fix Version/s 0.9 [ 11069 ]
Trustin Lee made changes - 29/Oct/05 12:08 AM
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]