LGTM,

Some lines are too long now.

One more thought: Maybe the code would look cleaner, if “getters” were spelled 
without the “get”, like “ofpbuf_size()”, “ofpbuf_l2()”. I know it would not 
rhyme with the “setters”, but as we do much less setting than getting it would 
still be cleaner? Thoughts?

  Jarno

Acked-by: Jarno Rajahalme <jrajaha...@nicira.com>

On Mar 31, 2014, at 9:43 PM, Pravin <pshe...@nicira.com> wrote:

> From: Pravin Shelar <pshe...@nicira.com>
> 
> These functions will be used by later patches.  Following patch
> does not change functionality.
> 
> Signed-off-by: Pravin B Shelar <pshe...@nicira.com>
...
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to