Date: Tue, 26 Nov 2002 11:56:20 -0500 (EST) From: John Baldwin <jhb@FreeBSD.org> To: "Greg 'groggy' Lehey" <grog@FreeBSD.org> Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, Julian Elischer <julian@FreeBSD.org>, Bruce Evans <bde@zeta.org.au> Subject: Re: cvs commit: src/sbin/fsck_ffs pass5.c Message-ID: <XFMail.20021126115620.jhb@FreeBSD.org> In-Reply-To: <20021126064832.GF49477@wantadilla.lemis.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 26-Nov-2002 Greg 'groggy' Lehey wrote: > On Tuesday, 26 November 2002 at 17:59:16 +1100, Bruce Evans wrote: >> On Tue, 26 Nov 2002, Greg 'groggy' Lehey wrote: >> >>> On Monday, 25 November 2002 at 17:00:03 +1100, Bruce Evans wrote: >>>> On Sun, 24 Nov 2002, Julian Elischer wrote: >>>> >>>>> julian 2002/11/24 13:41:42 PST >>>>> >>>>> Modified files: >>>>> sbin/fsck_ffs pass5.c >>>>> ... >>>> Please fix the style bugs (excessive parentheses) in this before merging. >>> >>> I thought there was consensus that additional parentheses were not >>> necessarily a Bad Thing if they served to demonstrate operator >>> precedence, as here. >> >> Here is a patch to fix these style bugs. >> >> ... > > I don't think anybody is in doubt about how to change it. > >> To me, parenthesizing almost eveything demonstrates that the author >> is not fluent in C. > > To me, parenthesizing demonstrates that the author thinks the reader > may not be fluent in C. If the reader is not fluent in C, the reader has many more problems to deal with than trying to figure out the order of operations when trying to read source code. This is FreeBSD, not CS 101. -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.20021126115620.jhb>