On Wed, Apr 2, 2014 at 12:43 PM, Jarno Rajahalme <jrajaha...@nicira.com> wrote: > 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? >
I agree, I updated patch and pushed it. > 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
