ofp-flow: Move parse_ofp_flow_mod_str() into correct file.

This function was left behind by accident in the patch that split up
ofp-parse.

Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Justin Pettit <jpettit@ovn.org>

diff --git a/lib/ofp-flow.c b/lib/ofp-flow.c
index af6be74909e6b5917a9649024876e7b8427326fc..10d6825991f677fb5cc3213fff686946f37e128c 100644 (file)
--- a/lib/ofp-flow.c
+++ b/lib/ofp-flow.c
@@ -1377,6 +1377,36 @@ parse_ofp_str(struct ofputil_flow_mod *fm, int command, const char *str_,
     return error;
 }
 
+/* Parses 'string' as an OFPT_FLOW_MOD or NXT_FLOW_MOD with command 'command'
+ * (one of OFPFC_*) into 'fm'.
+ *
+ * If 'command' is given as -2, 'string' may begin with a command name ("add",
+ * "modify", "delete", "modify_strict", or "delete_strict").  A missing command
+ * name is treated as "add".
+ *
+ * Returns NULL if successful, otherwise a malloc()'d string describing the
+ * error.  The caller is responsible for freeing the returned string. */
+char * OVS_WARN_UNUSED_RESULT
+parse_ofp_flow_mod_str(struct ofputil_flow_mod *fm, const char *string,
+                       const struct ofputil_port_map *port_map,
+                       const struct ofputil_table_map *table_map,
+                       int command,
+                       enum ofputil_protocol *usable_protocols)
+{
+    char *error = parse_ofp_str(fm, command, string, port_map, table_map,
+                                usable_protocols);
+
+    if (!error) {
+        /* Normalize a copy of the match.  This ensures that non-normalized
+         * flows get logged but doesn't affect what gets sent to the switch, so
+         * that the switch can do whatever it likes with the flow. */
+        struct match match_copy = fm->match;
+        ofputil_normalize_match(&match_copy);
+    }
+
+    return error;
+}
+
 /* Opens file 'file_name' and reads each line as a flow_mod of the specified
  * type (one of OFPFC_*).  Stores each flow_mod in '*fm', an array allocated
  * on the caller's behalf, and the number of flow_mods in '*n_fms'.

diff --git a/lib/ofp-parse.c b/lib/ofp-parse.c
index 4feb82ed54bb1b179733aa653609a23e7c727f5f..a8b5a877c59ef852cc22ad0d4b15c2bd1f0fc894 100644 (file)
--- a/lib/ofp-parse.c
+++ b/lib/ofp-parse.c
@@ -249,36 +249,6 @@ ofp_extract_actions(char *s)
         return NULL;
     }
 }
-
-/* Parses 'string' as an OFPT_FLOW_MOD or NXT_FLOW_MOD with command 'command'
- * (one of OFPFC_*) into 'fm'.
- *
- * If 'command' is given as -2, 'string' may begin with a command name ("add",
- * "modify", "delete", "modify_strict", or "delete_strict").  A missing command
- * name is treated as "add".
- *
- * Returns NULL if successful, otherwise a malloc()'d string describing the
- * error.  The caller is responsible for freeing the returned string. */
-char * OVS_WARN_UNUSED_RESULT
-parse_ofp_flow_mod_str(struct ofputil_flow_mod *fm, const char *string,
-                       const struct ofputil_port_map *port_map,
-                       const struct ofputil_table_map *table_map,
-                       int command,
-                       enum ofputil_protocol *usable_protocols)
-{
-    char *error = parse_ofp_str(fm, command, string, port_map, table_map,
-                                usable_protocols);
-
-    if (!error) {
-        /* Normalize a copy of the match.  This ensures that non-normalized
-         * flows get logged but doesn't affect what gets sent to the switch, so
-         * that the switch can do whatever it likes with the flow. */
-        struct match match_copy = fm->match;
-        ofputil_normalize_match(&match_copy);
-    }
-
-    return error;
-}
 \f
 static size_t
 parse_value(const char *s, const char *delimiters)