Re: [CFT] [JANITORIAL] Unbork fs.h

From: Andreas Dilger (
Date: Fri Jan 04 2002 - 17:15:13 EST

On Jan 04, 2002 09:45 -0700, Bryan Henderson wrote:
> >> sizeof (foo): 1611, sizeof(foo): 19364 => -bs should be removed
> >> ...
> >> int
> >> foo(int x): 11408, int foo(int x): 57275 => -psl should be removed
> >
> >I do not think good style is best defined by majority vote.
> I don't think the implication was that sizeof(foo) is better style because
> more people like it. The implication is that consistency is, in general,
> good programming style and it's easier to arrive at consistency by adhering
> to the majority style than by adhering to the minority style.

That was my goal.

> And I don't see what any of this has to do with whether an option should be
> removed from Lindent. Lindent should be a tool, which means it helps a
> user do whatever he wants to do. Whether he should want to do "sizeof
> (foo)" is a separate issue.

Well Lindent != indent. The "indent" program can do formatting to the
wishes of the user. However, "Lindent" is a wrapper script which is
trying to impose the will of Linus on other kernel programmers, and as
such "what the user wants to do" is of no concern. If they don't want
to follow the "mandated" coding style, then they just don't use Lindent.

Cheers, Andreas

Andreas Dilger

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to More majordomo info at Please read the FAQ at

This archive was generated by hypermail 2b29 : Mon Jan 07 2002 - 21:00:27 EST