Thanks for the reply,

My review of 06/07 patches is done.


On Mon, Jun 24, 2013 at 10:57 AM, Ben Pfaff <b...@nicira.com> wrote:

> On Fri, Jun 21, 2013 at 12:33:29PM -0700, Alex Wang wrote:
> > Looks good to me, just found two places where the line length exceeds 79,
>
> Thanks.  I fixed the one in our code and left the other one in sflow
> code alone (the line was already too long there and I don't like to
> mess with the sflow library).
>
> > Also, want to ask does it make sense the add line length check during
> > 'make'?
> >
> > e.g. the check can be as simple as:
> >
> > grep '.\{80\}' file
>
> So far, the rules that we check at make time are safety rules, not
> style rules.  I prefer to enforce style rules socially rather than
> mechanically.  (And occasionally wider code is more readable than
> narrower code.)
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to