On Thu, 24 Aug 2006 18:15:29 +0000, Al Stone wrote: > 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 :). I don't really like them much myself, but that is beside the point. I thought it was in the style guide, but if its not, I have no problem with them staying there. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |