lkml.org 
[lkml]   [2008]   [Apr]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch] checkpatch: relax spacing and line length

On Wednesday 2008-04-09 15:14, Andy Whitcroft wrote:
>On Wed, Apr 09, 2008 at 10:46:06AM +0200, Andi Kleen wrote:
>> > Oh, and if people felt that the concensus was for something to be
>> > implemented and that you are waiting for me to implement the change in
>> > checkpatch; please say so.
>>
>> Well at least I think the printk change is a good one to implement and there
>> wasn't much protest to it at least.
>
>Ok. will put this on my todo list.

Instead of

if (foo) {
if (baz) {
++x;
printk("Oh so long line makes my coding style go wary... nonsensical sentence\n");
}
}

I'd keep the indent and allow elongated lines:

if (foo) {
if (baz) {
++x;
printk("Oh so long line makes my coding style go wary... nonsensical sentence\n");
}
}

Or perhaps you just pointed out we need a smarter grep program! :)


\
 
 \ /
  Last update: 2008-04-09 15:21    [W:0.063 / U:0.612 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site