]> git.proxmox.com Git - mirror_ovs.git/commit
ofp-actions: Split ofpacts_check__() into many functions.
authorBen Pfaff <blp@ovn.org>
Fri, 15 Jun 2018 23:29:22 +0000 (16:29 -0700)
committerBen Pfaff <blp@ovn.org>
Tue, 31 Jul 2018 20:11:13 +0000 (13:11 -0700)
commitae6f7530bff39c62d3b7d1baab93623d8f5f3db1
tree0db01af5bfa256b1b7b4cfec1efe73a6b08649dd
parent86c6aa826e9274e5cb0e49159ba5e825c43065b2
ofp-actions: Split ofpacts_check__() into many functions.

ofpacts_check__() was a huge switch statement with special cases for many
different kinds of actions.  This made it unwieldy and put the special
cases far away from the rest of the code related to a given action.  This
commit refactors the code to avoid the problem.

Signed-off-by: Ben Pfaff <blp@ovn.org>
Tested-by: Yifeng Sun <pkusunyifeng@gmail.com>
Reviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>
include/openvswitch/ofp-actions.h
lib/ofp-actions.c
lib/ofp-flow.c
ofproto/ofproto-dpif-trace.c
ofproto/ofproto.c
utilities/ovs-ofctl.c