On Dec 20, 2013, at 8:41 AM, Ben Pfaff <[email protected]> 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 <[email protected]> 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 <[email protected]> >>> >>> Acked-by: Ben Pfaff <[email protected]> >> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
