From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5654 invoked by alias); 3 Jan 2011 13:00:22 -0000 Received: (qmail 5645 invoked by uid 22791); 3 Jan 2011 13:00:21 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from fencepost.gnu.org (HELO fencepost.gnu.org) (140.186.70.10) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 03 Jan 2011 13:00:16 +0000 Received: from eliz by fencepost.gnu.org with local (Exim 4.69) (envelope-from ) id 1PZk1B-0000fr-B0; Mon, 03 Jan 2011 08:00:09 -0500 From: Eli Zaretskii To: Joel Brobecker CC: msnyder@vmware.com, hjl.tools@gmail.com, mark.kettenis@xs4all.nl, gdb-patches@sourceware.org In-reply-to: <20110103043359.GO2396@adacore.com> (message from Joel Brobecker on Mon, 3 Jan 2011 08:33:59 +0400) Subject: Re: [patch] more comment cleanups Reply-to: Eli Zaretskii References: <4D1E60A0.1020601@vmware.com> <201012312312.oBVNC4fc013647@glazunov.sibelius.xs4all.nl> <4D1E732E.4090002@vmware.com> <834o9tq96f.fsf@gnu.org> <4D1F71B0.9060006@vmware.com> <20110103043359.GO2396@adacore.com> Message-Id: Date: Mon, 03 Jan 2011 13:00:00 -0000 X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2011-01/txt/msg00039.txt.bz2 > Date: Mon, 3 Jan 2011 08:33:59 +0400 > From: Joel Brobecker > Cc: Eli Zaretskii , "hjl.tools@gmail.com" , "mark.kettenis@xs4all.nl" , "gdb-patches@sourceware.org" > > > So what do others think about source lines over 70 columns? > > Enforcing a maximum length of 70 characters seems awfully low to me. > It's fine if someone decides to use 70 characters for his comments, > of even his code, but I think that 76 or 78 would give better results. If we go for a higher number and want to enforce it (and I'm not saying we should), we should try to add a suitable setting to the C source files (in the local variables section), or maybe provide a .dir-locals.el file, for the benefit of Emacs users. Of course, not everyone uses Emacs.