From owner-freebsd-hackers@FreeBSD.ORG Sun May 3 07:11:17 2009 Return-Path: Delivered-To: freebsd-hackers@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 192D41065674 for ; Sun, 3 May 2009 07:11:17 +0000 (UTC) (envelope-from christoph.mallon@gmx.de) Received: from mail.gmx.net (mail.gmx.net [213.165.64.20]) by mx1.freebsd.org (Postfix) with SMTP id 2C4BE8FC14 for ; Sun, 3 May 2009 07:11:15 +0000 (UTC) (envelope-from christoph.mallon@gmx.de) Received: (qmail invoked by alias); 03 May 2009 07:11:14 -0000 Received: from p54A3DFED.dip.t-dialin.net (EHLO tron.homeunix.org) [84.163.223.237] by mail.gmx.net (mp062) with SMTP; 03 May 2009 09:11:14 +0200 X-Authenticated: #1673122 X-Provags-ID: V01U2FsdGVkX1/+GocFGSb+8rHm9duOasvX86IrLHmYoWlKayL1DN IBRRJjYO+vutNb Message-ID: <49FD4391.9070605@gmx.de> Date: Sun, 03 May 2009 09:11:13 +0200 From: Christoph Mallon User-Agent: Thunderbird 2.0.0.21 (X11/20090412) MIME-Version: 1.0 To: "M. Warner Losh" References: <49FCA148.9060707@gmx.de> <49FCAA1D.1080208@elischer.org> <49FCAFA2.60603@gmx.de> <20090502.151931.1396014860.imp@bsdimp.com> In-Reply-To: <20090502.151931.1396014860.imp@bsdimp.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Y-GMX-Trusted: 0 X-FuHaFi: 0.57 Cc: sobomax@FreeBSD.org, freebsd-hackers@FreeBSD.org, rdivacky@FreeBSD.org, ed@FreeBSD.org, dwmalone@maths.tcd.ie, julian@elischer.org Subject: Re: C99: Suggestions for style(9) X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 03 May 2009 07:11:17 -0000 M. Warner Losh schrieb: > In message: <49FCAFA2.60603@gmx.de> > Christoph Mallon writes: > : Julian Elischer schrieb: > : >> Christoph Mallon wrote: > : >>>> K&R code should be changed as part of related changes if possible. > : >>>> A sweep to change a whole file is probably also ok. > : >>>> changing them one at a time is probably not ok. > : >>> > : >>> But this is what actually is practiced. > : >>> You still did not answer my question: Do you agree to remove the > : >>> clause so no new old style declarations may be added? > : > > : > I think a new clause should be added specifying what should happen > : > and replacing the old clause. > : > : This is not sensible. style(9) says right at the start that it "[...] > : specifies the preferred style for kernel source files [...]". The > : preferred style would be to use ANSI function declarations - what else > : is there to say? There is no point in adding more when less is sufficient. > > Actually, in a style guide, there is a point. > > Adding language that says we're actively removing K&R-style > declarations and definitions reinforces this point and explains to > people what's going on when they see this in the tree today. This just overcomplicates things. "removing old style definitions" is not the preferred style, but "using prototyped definitions" is. Old style definitions should not be added anymore, so just remove the clause, which allows it currently. Adding even more about old style definitions is counterproductive - I cannot support this. What to do, when you are seeing an old style definition is clear: Don't Panic! Christoph