On Thu, Dec 21, 2017 at 04:39:04PM -0800, Justin Pettit wrote:
> 
> 
> > On Nov 27, 2017, at 1:53 PM, Ben Pfaff <[email protected]> wrote:
> > 
> > This function uses local array set_buff[] to store Netlink attributes.
> > It declares set_buff as an array of character pointers, which is a strange
> > type for a buffer of non-character-pointer objects.  In OVS it is
> > customary to use an ofpbuf with a stub of uint64_t objecs (to ensure
> > proper alignment, otherwise uint8_t would be more usual).  This commit
> > changes to that more usual form.
> > 
> > Signed-off-by: Ben Pfaff <[email protected]>
> 
> Acked-by: Justin Pettit <[email protected]>

Thanks, applied to master.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to