From: Joe Stringer <joe@wand.net.nz>
Date: Wed, 11 Jul 2012 23:07:15 +0000 (+1200)
Subject: meta-flow: Reduce redundant transport set/get code
X-Git-Tag: sliver-openvswitch-1.8.90-0~48^2~261
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=71baec0655e41bb93f125d162bc62cc9da0be73b;p=sliver-openvswitch.git

meta-flow: Reduce redundant transport set/get code

Signed-off-by: Joe Stringer <joe@wand.net.nz>
Signed-off-by: Ben Pfaff <blp@nicira.com>
---

diff --git a/lib/meta-flow.c b/lib/meta-flow.c
index 8e7d4fc8a..8ba21fc11 100644
--- a/lib/meta-flow.c
+++ b/lib/meta-flow.c
@@ -998,17 +998,11 @@ mf_get_value(const struct mf_field *mf, const struct flow *flow,
         break;
 
     case MFF_TCP_SRC:
-        value->be16 = flow->tp_src;
-        break;
-
-    case MFF_TCP_DST:
-        value->be16 = flow->tp_dst;
-        break;
-
     case MFF_UDP_SRC:
         value->be16 = flow->tp_src;
         break;
 
+    case MFF_TCP_DST:
     case MFF_UDP_DST:
         value->be16 = flow->tp_dst;
         break;
@@ -1143,17 +1137,11 @@ mf_set_value(const struct mf_field *mf,
         break;
 
     case MFF_TCP_SRC:
-        cls_rule_set_tp_src(rule, value->be16);
-        break;
-
-    case MFF_TCP_DST:
-        cls_rule_set_tp_dst(rule, value->be16);
-        break;
-
     case MFF_UDP_SRC:
         cls_rule_set_tp_src(rule, value->be16);
         break;
 
+    case MFF_TCP_DST:
     case MFF_UDP_DST:
         cls_rule_set_tp_dst(rule, value->be16);
         break;