Signed-off-by: Isaku Yamahata <[email protected]>
---
 ryu/ofproto/ofproto_v1_0_parser.py |   19 ++++++++++++++-----
 1 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/ryu/ofproto/ofproto_v1_0_parser.py 
b/ryu/ofproto/ofproto_v1_0_parser.py
index 4563807..067d28a 100644
--- a/ryu/ofproto/ofproto_v1_0_parser.py
+++ b/ryu/ofproto/ofproto_v1_0_parser.py
@@ -743,9 +743,18 @@ class NXTSetFlowFormat(NXTRequest):
 
 
 class NXTFlowMod(NXTRequest):
-    def __init__(self, datapath, cookie, command, idle_timeout,
-                 hard_timeout, priority, buffer_id, out_port,
-                 flags, rule, actions=None):
+    def __init__(self, datapath, cookie, command,
+                 idle_timeout=0, hard_timeout=0,
+                 priority=ofproto_v1_0.OFP_DEFAULT_PRIORITY,
+                 buffer_id=0xffffffff, out_port=ofproto_v1_0.OFPP_NONE,
+                 flags=0, rule=None, actions=None):
+
+        # the argument, rule, is positioned at the one before the last due
+        # to the layout struct nxt_flow_mod.
+        # Although rule must be given, default argument to rule, None,
+        # is given to allow other default value of argument before rule.
+        assert rule is not None
+
         if actions is None:
             actions = []
         super(NXTFlowMod, self).__init__(datapath, ofproto_v1_0.NXT_FLOW_MOD)
@@ -1155,8 +1164,8 @@ class OFPPacketOut(MsgBase):
 
 @_set_msg_type(ofproto_v1_0.OFPT_FLOW_MOD)
 class OFPFlowMod(MsgBase):
-    def __init__(self, datapath, match, cookie=0,
-                 command=0, idle_timeout=0, hard_timeout=0,
+    def __init__(self, datapath, match, cookie, command,
+                 idle_timeout=0, hard_timeout=0,
                  priority=ofproto_v1_0.OFP_DEFAULT_PRIORITY,
                  buffer_id=0xffffffff, out_port=ofproto_v1_0.OFPP_NONE,
                  flags=0, actions=None):
-- 
1.7.1.1


------------------------------------------------------------------------------
Better than sec? Nothing is better than sec when it comes to
monitoring Big Data applications. Try Boundary one-second 
resolution app monitoring today. Free.
http://p.sf.net/sfu/Boundary-dev2dev
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to