[ovs-dev] [PATCH v2] ofp-actions: Split ofpacts_check__() into many functions.

Ben Pfaff blp at ovn.org
Thu Jul 12 23:33:51 UTC 2018


On Fri, Jun 15, 2018 at 04:29:22PM -0700, Ben Pfaff wrote:
> 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 at ovn.org>
> ---
> I'm reposting this in hope of getting a review this time.

Still needs review.


More information about the dev mailing list