lkml.org 
[lkml]   [2012]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch cr 2/4] [RFC] syscalls, x86: Add __NR_kcmp syscall v7
On Fri, 3 Feb 2012 10:09:29 +0100 Ingo Molnar <mingo@elte.hu> wrote:

>
> * Cyrill Gorcunov <gorcunov@openvz.org> wrote:
>
> > > > + get_random_bytes(&cookies[i][j],
> > > > + sizeof(cookies[i][j]));
> > >
> > > ugly line break.
> > >
> >
> > Why? Looks pretty good to me. But sure I'll change it.
>
> It's ugly because it serves no purpose other than pacifying
> checkpatch and makes the code *uglier*.

No it doesn't. For 80-col displays the code is *already wrapped*. And
that wrapping to column 0 is vastly worse than the above.

If we want to increase the standard to (say) 96 cols then fine, I'd be
happy with that. But until we do that we should not create such a
gruesome mess for those who use 80 cols.

> It's a disease. When checkpatch tells you "this line is too
> long" then consider it a code cleanliness warning!

Well yes, if it can be fixed by other means then great.



\
 
 \ /
  Last update: 2012-02-03 10:25    [W:0.072 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site