[ovs-dev] [PATCH 13/24] ofp-util: Allow encoding of Open Flow 1.2 Flow Mod messages

Simon Horman horms at verge.net.au
Wed Jul 25 06:04:24 UTC 2012


On Tue, Jul 24, 2012 at 10:40:17PM -0700, Ben Pfaff wrote:
> On Mon, Jul 23, 2012 at 03:16:42PM +0900, Simon Horman wrote:
> > Signed-off-by: Simon Horman <horms at verge.net.au>
> 
> Looks good, thanks.
> 
> I think the memset() in the new OFPUTIL_P_OF12 case is unnecessary
> because you use ofpbuf_put_zeros() before that.

Thanks, I'll fix that. I think it is there because originally
ofpbuf_put_zeros() wasn't used.



More information about the dev mailing list