RE: [PATCH] 80-column reformatting for fsys.S

From: Al Stone <>
Date: 2006-08-25 04:15:29
On Thu, 2006-08-24 at 09:09 -0700, Luck, Tony wrote:
> > > Could you convert the C++ comments to C comments while you are at it?
> >
> > Good idea.  I'll try to make a second pass on the ones I've done,
> > and make it a point to do it in the future.
> I'm not so sure that this is such a great idea ... using C comments will
> trim another 2 or three characters off the space you have available
> on each line (depending on whether you like /*comment*/ or /* comment */)
> so you will either need to be more terse in order to fit, or have more
> comments run onto two lines, neither of which sound like an improvememt
> to maintainability).

I'm not personally committed to either style of comment.  I agree
that it could compress things in a bad way to use C-style.  And I'm
not really that particular that they all need to be one style or the
other (both seem to be used intermittently within the same file).
I'll gladly bow to the majority, whichever it is.

> The 80-column rule has considerable merit (I like to have two code
> windows side-by-side, and my employer has shown no signs of wanting
> to shell out for a 24+" monitor for me :-( ... but I'm really not
> so sure why people are so anti-c++ comments {BTW, I don't see this
> prohibition in Documentation/CodingStyle ... is this just another
> unwritten rule that everyone knows about?)

Heh.  Indeed.  As my eyes get older, the monitors seem to have to get
larger :).  And hence the 80-column cleanup.

I couldn't find comment _style_ mentioned in CodingStyle either.  The
only reason I know of for using C-style comments is to maintain some
consistency with C; logically, that makes _some_ sense, but again, I
personally don't care that much one way or the other.  I'm willing
to go with whatever gets accepted as a patch :).

Al Stone                                      Alter Ego:
Open Source and Linux R&D                     Debian Developer
Hewlett-Packard Company             

To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to
More majordomo info at
Received on Fri Aug 25 04:16:27 2006

This archive was generated by hypermail 2.1.8 : 2006-08-25 04:16:38 EST