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

[Andreas Dilger <adilger@turbolabs.com>]

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
http://sourceforge.net/projects/ext2resize/
http://www-mddsp.enel.ucalgary.ca/People/adilger/