From 863d2e1a8c2a6ced49a49024c094ef6a9aa7e55a Mon Sep 17 00:00:00 2001 From: Adrian Moreno Date: Mon, 19 Dec 2022 17:13:52 +0100 Subject: python: Don't exit OFPFlow constructor. Returning None in a constructor does not make sense and is just error prone. Removing what was a leftover 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 Acked-by: Mike Pattrick Signed-off-by: Ilya Maximets --- python/ovs/flow/ofp.py | 3 --- 1 file changed, 3 deletions(-) (limited to 'python/ovs') 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: -- cgit v1.2.1