lkml.org 
[lkml]   [2007]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] run scripts/Lindent on it to match Documentation/CodingStyle
On Sun, Jul 15, 2007 at 10:53:45AM +0100, Simon Arlott wrote:
> > - } else if (base_addr > 0x100) { /* Check a single specified location. */
> > + } else if (base_addr > 0x100) { /* Check a single specified location. */
>
> What is Lit doing here?! It's changed "{<space>/*" to "{<tab>/*"...
>
> > - } else { /* Scan all possible addresses of the WaveLAN hardware. */
> > + } else { /* Scan all possible addresses of the WaveLAN hardware. */
>
> And again... with two tabs for maximum unreadability. That line is now
> 90 characters long instead of 75.

There's two things going on here. One is that we haven't told indent to
cram the comments after code up against the code -- by default it will
move the comment out to start in column 33. We can override that by
adding '-c1' to the Lindent line.

The other is that, even when you do that, it only wants to indent
comments with tabs. So in this example:

int bar(void)
{
if (x) {
} else if (quite_a_long_conditional_which) { /* takes up a lot of ram */
} else { /* An absolutely gargantuan comment that heads to the end */
} /* Another comment */
}

by default Lindent will move the comment to:

} else if (quite_a_long_conditional_which) { /* takes up a lot of ram */
} else { /* An absolutely gargantuan comment that heads to the end */
} /* Another comment */

adding -c1 gets us:

} else if (quite_a_long_conditional_which) { /* takes up a lot of ram */
} else { /* An absolutely gargantuan comment that heads to the end */
} /* Another comment */

but the indent manpage is quite definite:

If the code to the left of the comment exceeds the beginning column,
the comment column will be extended to the next tabstop column past the
end of the code, or in the case of preprocessor directives, to one
space past the end of the directive.

I suppose someone could add a new option to indent to change that, but
I'd rather see people not put comments there, tbh.

> > - "%s: <-wavelan_probe()\n",
> > - dev->name);
> > + "%s: <-wavelan_probe()\n", dev->name);
>
> There are spaces in that line after the tabs...

I think that's to make it line up with the beginning of the function arguments.
Can't tell cos you snipped that bit ;-)

--
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-07-20 19:37    [W:0.097 / U:0.696 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site