On Jun 5, 2013, at 4:42 PM, Ben Pfaff <b...@nicira.com> wrote: > On Wed, Jun 05, 2013 at 03:21:37PM -0700, Justin Pettit wrote: >> From: Ethan Jackson <et...@nicira.com> >> >> This function will be useful in a future commit. >> >> Co-authored-by: Justin Pettit <jpet...@nicira.com> >> Signed-off-by: Justin Pettit <jpet...@nicira.com> > > The name flow_wildcards_or_miniflow() threw me for a loop until I read > the function carefully. Could you add a function-level comment? > Actually, the prefix "flow_wildcards" itself is a little confusing > since the function doesn't use a flow_wildcards anywhere (and > miniflow_expand() doesn't apply it to a flow_wildcards).
How about "flow_union_with_miniflow" and the following comment? /* Perform a bit-wise OR of miniflow 'src' flow data with the equivilent * fields in 'dst', storing the result in 'dst'. */ --Justin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
