From: Justin Pettit <jpettit@nicira.com>
Date: Tue, 12 Oct 2010 06:50:36 +0000 (-0700)
Subject: odp-util: Remove extraneous "set_tci(" from format_odp_action()
X-Git-Tag: v1.1.0~1005
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=db164669c1b482843e082e752d8fbd69f650f94a;p=sliver-openvswitch.git

odp-util: Remove extraneous "set_tci(" from format_odp_action()

The function format_odp_action(), which is used to format the output of
"ovs-dpctl dump-flows", printed "set_tci(" before printing a correct
"set_tci" with actual parameters.

Bug #3808 (1/2)
---

diff --git a/lib/odp-util.c b/lib/odp-util.c
index b497538d5..130d55de7 100644
--- a/lib/odp-util.c
+++ b/lib/odp-util.c
@@ -75,7 +75,6 @@ format_odp_action(struct ds *ds, const union odp_action *a)
         int vid = vlan_tci_to_vid(a->dl_tci.tci);
         int pcp = vlan_tci_to_pcp(a->dl_tci.tci);
 
-        ds_put_cstr(ds, "set_tci(");
         switch (ntohs(a->dl_tci.mask)) {
         case VLAN_VID_MASK:
             ds_put_format(ds, "set_tci(vlan=%d)", vid);