On Dec 20, 2013, at 8:41 AM, Ben Pfaff <b...@nicira.com> wrote: > I see that the commit as you pushed it accidentally includes a changelog > in the commit message. That's not usual practice; careful. >
Oops, sorry. Is it still possible to remove it? Jarno > 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
