cleaning

@@ -65,7 +65,6 @@ import org.slf4j.LoggerFactory;
  */
 class OFChannelHandler extends IdleStateAwareChannelHandler {
     private static final Logger log = LoggerFactory.getLogger(OFChannelHandler.class);
-    private static final long DEFAULT_ROLE_TIMEOUT_MS = 2 * 1000; // 10 sec
     private final Controller controller;
     private AbstractOpenFlowSwitch sw;
     private long thisdpid; // channelHandler cached value of connected switch id

 package org.onlab.onos.of.drivers;
 
 
-import java.util.List;
 
 import org.onlab.onos.of.controller.Dpid;
 import org.onlab.onos.of.controller.RoleState;
@@ -56,52 +55,39 @@ public final class DriverManager {
                 + "Assigning OFSwitchImplBase", desc);
         AbstractOpenFlowSwitch base = new AbstractOpenFlowSwitch(dpid) {
 
-            @Override
-            public void write(List<OFMessage> msgs) {
-                // TODO Auto-generated method stub
-            }
 
             @Override
             public void sendMsg(OFMessage m) {
-                // TODO Auto-generated method stub
+                channel.write(m);
             }
 
             @Override
             public Boolean supportNxRole() {
-                // TODO Auto-generated method stub
-                return null;
+                return false;
             }
 
             @Override
-            public void startDriverHandshake() {
-                // TODO Auto-generated method stub
-            }
+            public void startDriverHandshake() {}
 
             @Override
             public void setFeaturesReply(OFFeaturesReply featuresReply) {
-                // TODO Auto-generated method stub
-
+                this.features = featuresReply;
             }
 
             @Override
-            public void processDriverHandshakeMessage(OFMessage m) {
-                // TODO Auto-generated method stub
-            }
+            public void processDriverHandshakeMessage(OFMessage m) {}
 
             @Override
             public boolean isDriverHandshakeComplete() {
-                // TODO Auto-generated method stub
-                return false;
+                return true;
             }
 
             @Override
             public RoleState getRole() {
-                // TODO Auto-generated method stub
-                return null;
+                return role;
             }
         };
         base.setSwitchDescription(desc);
-        // XXX S must set counter here - unidentified switch
         return base;
     }