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/06/20 04:53:08 UTC
svn commit: r548901 -
/mina/branches/1.0/core/src/main/java/org/apache/mina/common/support/DefaultIoFuture.java
Author: trustin
Date: Tue Jun 19 19:53:07 2007
New Revision: 548901
URL: http://svn.apache.org/viewvc?view=rev&rev=548901
Log:
Fixed DefaultIoFuture.removeListener doesn't remove the first listener when there's only one listener
Modified:
mina/branches/1.0/core/src/main/java/org/apache/mina/common/support/DefaultIoFuture.java
Modified: mina/branches/1.0/core/src/main/java/org/apache/mina/common/support/DefaultIoFuture.java
URL: http://svn.apache.org/viewvc/mina/branches/1.0/core/src/main/java/org/apache/mina/common/support/DefaultIoFuture.java?view=diff&rev=548901&r1=548900&r2=548901
==============================================================================
--- mina/branches/1.0/core/src/main/java/org/apache/mina/common/support/DefaultIoFuture.java (original)
+++ mina/branches/1.0/core/src/main/java/org/apache/mina/common/support/DefaultIoFuture.java Tue Jun 19 19:53:07 2007
@@ -210,6 +210,8 @@
if (listener == firstListener) {
if (otherListeners != null && !otherListeners.isEmpty()) {
firstListener = (IoFutureListener) otherListeners.remove(0);
+ } else {
+ firstListener = null;
}
} else if (otherListeners != null) {
otherListeners.remove(listener);