dpif: Factor 'type' and 'error' out of individual dpif_op members.
authorBen Pfaff <blp@nicira.com>
Mon, 26 Dec 2011 22:17:55 +0000 (14:17 -0800)
committerBen Pfaff <blp@nicira.com>
Mon, 16 Jan 2012 21:35:21 +0000 (13:35 -0800)
commitc2b565b54e36bc33d0406afb225c9bf3d01405ef
tree9861ebc77f2a857bf62c4ae43fa6231384ed10d6
parent8338659aa5a02421959bece64f6233216de6a101
dpif: Factor 'type' and 'error' out of individual dpif_op members.

I'd like to change ->dpif_flow_put() and ->dpif_execute() in the dpif
provider to take the structures of the same names as parameters, instead of
passing them discrete parameters, because this seems like a more sensible
way to do things internally than to have two different ways to pass the
parameters.  It might even simplify code slightly.  But ->flow_put() and
->execute() wouldn't want the 'type' (because it's implied by the function
being called) or 'error' (because it would be the same as the return
value).  Although of course they could just ignore those members, it seems
slightly cleaner to omit them entirely, as this change allows.

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/dpif-linux.c
lib/dpif-provider.h
lib/dpif.c
lib/dpif.h
ofproto/ofproto-dpif.c