On Sat, Jan 07, 2012 at 11:11:46AM -0800, Ethan Jackson wrote:
> In future patches, directly using a void * pointer in the
> pktbuf_save() definition will simplify the code.
> 
> Signed-off-by: Ethan Jackson <et...@nicira.com>

In pktbuf_save(), you can use ofpbuf_clone_data_with_headroom() instead
of open-coding it.

I think there's an existing incorrect statement in the comment on
pktbuf_save(): 'in_port' should be an OpenFlow port number these days, I
think.

Otherwise it looks good.  Thank you.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to