[ovs-dev] [PATCH ovn 2/2] northd: Remove unused match and actions strings.

Mark Michelson mmichels at redhat.com
Tue Jun 15 13:24:44 UTC 2021


These strings are initialized and destroyed, but they aren't used
otherwise. This commit simply removes them.

Signed-off-by: Mark Michelson <mmichels at redhat.com>
---
 northd/ovn-northd.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c
index d872f6a3c..a6cea285b 100644
--- a/northd/ovn-northd.c
+++ b/northd/ovn-northd.c
@@ -6844,8 +6844,6 @@ build_lswitch_flows(struct hmap *datapaths, struct hmap *lflows)
     /* This flow table structure is documented in ovn-northd(8), so please
      * update ovn-northd.8.xml if you change anything. */
 
-    struct ds match = DS_EMPTY_INITIALIZER;
-    struct ds actions = DS_EMPTY_INITIALIZER;
     struct ovn_datapath *od;
 
     /* Ingress table 23: Destination lookup for unknown MACs (priority 0). */
@@ -6868,9 +6866,6 @@ build_lswitch_flows(struct hmap *datapaths, struct hmap *lflows)
         ovn_lflow_add(lflows, od, S_SWITCH_IN_L2_UNKNOWN, 0, "1",
                       "output;");
     }
-
-    ds_destroy(&match);
-    ds_destroy(&actions);
 }
 
 /* Build pre-ACL and ACL tables for both ingress and egress.
-- 
2.31.1



More information about the dev mailing list