Thanks for the review, pushed with the change suggested, Jarno
On May 14, 2014, at 10:30 AM, Ben Pfaff <b...@nicira.com> wrote: > On Mon, May 12, 2014 at 11:21:11PM -0700, Jarno Rajahalme wrote: >> The terminology we used for subtable ordering ('splice', 'right >> before') was inherited from an earlier use of a linked list, and >> turned out to be confusing when applied to an array. Also, we only >> ever move one subtable earlier or later within the array, so we can >> simplify the code as well. >> >> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > Each of the callers has an explicit test for to != from. Can we just > move that into cls_subtable_cache_move()? > > Acked-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev