returning None in a constructor does not make sense and is just error
prone. Removing what was a lefover from an attempt to handle a common
error case of trying to parse what is commonly outputted by ovs-ofctl.
This should be done by the caller anyway.
Signed-off-by: Adrian Moreno <amore...@redhat.com>
Acked-by: Mike Pattrick <m...@redhat.com>
---
python/ovs/flow/ofp.py | 3 ---
1 file changed, 3 deletions(-)
diff --git a/python/ovs/flow/ofp.py b/python/ovs/flow/ofp.py
index eac8d0851..20231fd9f 100644
--- a/python/ovs/flow/ofp.py
+++ b/python/ovs/flow/ofp.py
@@ -104,9 +104,6 @@ class OFPFlow(Flow):
ValueError if the string is malformed.
ParseError if an error in parsing occurs.
"""
- if " reply " in ofp_string:
- return None
-
sections = list()
parts = ofp_string.split("actions=")
if len(parts) != 2:
--
2.38.1
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
