Hi,

Committed. This fixes the bug that when a key was already registered it
would not get the new interest ops and the attachment wouldn't be
cleared if the value value was null.

Regards,
Jeroen

2006-09-24  Jeroen Frijters  <[EMAIL PROTECTED]>

        * java/nio/channels/spi/AbstractSelectableChannel.java
        (register): Set interestOps and attachment when the key already
        exists.
Index: java/nio/channels/spi/AbstractSelectableChannel.java
===================================================================
RCS file: 
/cvsroot/classpath/classpath/java/nio/channels/spi/AbstractSelectableChannel.java,v
retrieving revision 1.20
diff -u -r1.20 AbstractSelectableChannel.java
--- java/nio/channels/spi/AbstractSelectableChannel.java        24 Sep 2006 
11:37:21 -0000      1.20
+++ java/nio/channels/spi/AbstractSelectableChannel.java        24 Sep 2006 
18:17:07 -0000
@@ -243,8 +243,8 @@
 
        if (key != null && key.isValid())
          {
-           if (att != null)
-             key.attach(att);
+            key.interestOps(ops);
+            key.attach(att);
          }
        else
          {

Reply via email to