Series pushed, thanks for the reviews! Jarno
On Aug 22, 2014, at 11:15 AM, Ben Pfaff <[email protected]> wrote: > On Fri, Aug 15, 2014 at 04:38:46PM -0700, Jarno Rajahalme wrote: >> "xlate_receive" did not tell much about what it is used for. We have >> two users of it that only want the ofproto and the OF port number, use the >> new xlate_lookup_ofproto() for those. >> >> Fix the comments of xlate_receive() as we no longer change the flow. >> >> Also, the helper ofproto_dpif_contains_flow() seemed ill-named, so this >> path removes it and uses xlate_lookup_ofproto() directly. >> >> Signed-off-by: Jarno Rajahalme <[email protected]> > > I think that with some small adjustments, xlate_receive() could call > xlate_lookup_ofproto() and avoid a small amount of duplicated code. > > Acked-by: Ben Pfaff <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
