From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9973 invoked by alias); 20 Nov 2004 18:00:39 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 9917 invoked from network); 20 Nov 2004 18:00:34 -0000 Received: from unknown (HELO balder.inter.net.il) (192.114.186.15) by sourceware.org with SMTP; 20 Nov 2004 18:00:34 -0000 Received: from zaretski ([80.230.142.118]) by balder.inter.net.il (Mirapoint Messaging Server MOS 3.3.7-GR) with ESMTP id DWI20979 (AUTH halo1); Sat, 20 Nov 2004 20:00:04 +0200 (IST) Date: Sat, 20 Nov 2004 18:00:00 -0000 From: "Eli Zaretskii" To: Mark Kettenis Message-ID: <01c4cf2a$Blat.v2.2.2$7bf2c3e0@zahav.net.il> Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=ISO-8859-1 CC: gdb-patches@sources.redhat.com In-reply-to: <200411201741.iAKHf0CV001472@elgar.sibelius.xs4all.nl> (message from Mark Kettenis on Sat, 20 Nov 2004 18:41:00 +0100 (CET)) Subject: Re: [COMMIT] Improve coding standard in dbxread.c Reply-to: Eli Zaretskii References: <200411201239.iAKCd9O5061792@elgar.sibelius.xs4all.nl> <01c4cf25$Blat.v2.2.2$44ec6860@zahav.net.il> <200411201741.iAKHf0CV001472@elgar.sibelius.xs4all.nl> X-SW-Source: 2004-11/txt/msg00407.txt.bz2 > Date: Sat, 20 Nov 2004 18:41:00 +0100 (CET) > From: Mark Kettenis > CC: gdb-patches@sources.redhat.com > > The coding standards say: > > "Also, please write complete sentences and capitalize the first word." standards.texi also has this example: #ifdef foo ... #else /* not foo */ ... #endif /* not foo */ #ifdef foo ... #endif /* foo */ where the comments are clearly not full sentences and don't end with a period. > Now indeed it is debatable whether the comments above are complete > sentences; there's no verb in them. But these comments feel very > sentence-like. I'm okay with making them full sentences, if you think it's better than what I suggested. But please let's not do half a job; let's at least have correct English in comments. What's there now is not correct English. Before your changes, we at least could say no one cared to fix those comments, but now we don't have that excuse anymore ;-). > Anyway, I was aiming for some consistency here. Consistency is important, but correct English is also important. > Some of the other changes simply reformat comments to break the line > at a different column. Do we have a canonical column number for that, > and if we do, what is its value? > > I always consider the GNU indentation style provided by emacs to be an > implementation of the canonical formatting style. Emacs lets you customize the column where it breaks a line that is too long. While your setup seems to use the default value, someone else might not, or could use something other than Emacs. So if we want to have consistent formatting, let's agree on where long lines should be wrapped. > The canonical column therefore would be 32. That's where comments start; I meant the column where they are wrapped. The default is 70 (zero-based).