From 10fa798590e27142f70cf7fa35a48bb50e540de1 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori Date: Thu, 30 Aug 2012 08:22:56 +0900 Subject: [PATCH] remove unnecessary OFPEchoReply handler in dispatchers We don't send OFPEchoRequest so there is no point to have OFPEchoReply handler. I think that sending OFPEchoRequest is useful for some cases (e.g. finding problems on the switch quickly). But that should be another helper dispatcher since not all applications need such. Signed-off-by: FUJITA Tomonori --- ryu/controller/handler.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/ryu/controller/handler.py b/ryu/controller/handler.py index 09b484bc..227f1706 100644 --- a/ryu/controller/handler.py +++ b/ryu/controller/handler.py @@ -117,14 +117,6 @@ class EchoHandler(object): echo_reply.data = msg.data datapath.send_msg(echo_reply) - @staticmethod - @set_ev_cls(ofp_event.EventOFPEchoReply) - def echo_reply_handler(ev): - # do nothing - # msg = ev.msg - # LOG.debug('echo reply ev %s %s', msg, str(msg.data)) - pass - @register_cls([HANDSHAKE_DISPATCHER, CONFIG_DISPATCHER, MAIN_DISPATCHER]) class ErrorMsgHandler(object):