I see that the commit as you pushed it accidentally includes a changelog in the commit message. That's not usual practice; careful.
On Fri, Dec 20, 2013 at 08:41:02AM -0800, Jarno Rajahalme wrote: > Pushed, thanks for the review! > > Jarno > > On Dec 19, 2013, at 1:42 PM, Ben Pfaff <b...@nicira.com> wrote: > > > On Thu, Dec 19, 2013 at 01:02:42PM -0800, Jarno Rajahalme wrote: > >> We allow zero 'values' in a miniflow for it to have the same map > >> as the corresponding minimask. Minimasks themselves never have > >> zero data values, though. Document this and optimize the code > >> accordingly. > >> > >> v2: > >> - Made miniflow_get_map_in_range() to return data offset instead of > >> a pointer via the last parameter. > >> - Simplified minimatch_hash_in_range() by removing pointer arithmetic. > >> > >> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > > > Acked-by: Ben Pfaff <b...@nicira.com> > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
