Re: [GIT PULL] percpu fixes for 2.6.32-rc6

From: Andres Baldrich
Date: Thu Nov 12 2009 - 12:05:12 EST




El jue 12-nov-09, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> escribió:
> On Thu, 12 Nov 2009, Tejun Heo wrote:
> >
> > 11/12/2009 07:36 PM, Ingo Molnar wrote:
> > >
> > > Breaking strings mid-sentence is something we try
> not to do. (If you
> > > know about places that do it 'quite often' then
> those places need fixing
> > > too.)
> >
> > Oh... I do that all the time and I see a lot of them
> around too.
>
> I hate them. I do greps for error messages, and it's
> annoying as hell if
> it's hard to find.
>
> 'checkpatch' is the major reason for them, but I think
> we've fixed
> checkpath long ago to not warn about long lines if they are
> due to a long
> string.
>
> Strings should basically be broken up only at '\n'
> characters, so
>
>     printk("This is a made-up example.\n"
>         "Ok like this\n");
>
> is fine, because you can expect to grep for "made-up
> example", but not
> over the newline.
>
>            
>     Linus


(Kernel)/Documentation/CodingStyle
line 83:
"Statements longer than 80 columns will be broken into sensible chunks.
Descendants are always substantially shorter than the parent and are placed
substantially to the right. The same applies to function headers with a long
argument list. Long strings are as well broken into shorter strings. The
only exception to this is where exceeding 80 columns significantly increases
readability and does not hide information.

void fun(int a, int b, int c)
{
if (condition)
printk(KERN_WARNING "Warning this is a long printk with "
"3 parameters a: %u b: %u "
"c: %u \n", a, b, c);
else
next_statement;
}"

Perhaps it should be changed?
I read the list's faq, it asked useful code or "diff" outputs for proposed changes, but i am an *absolute* beginner with < 6 months of college. 'Never' used FOSS before.
I hope it was good imput.
Andrés Baldrich <abaldric@xxxxxxxxxxxxxxx>


Yahoo! Cocina

Encontra las mejores recetas con Yahoo! Cocina.


http://ar.mujer.yahoo.com/cocina/

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/