Date: Wed, 20 Apr 2016 09:33:07 -0500 From: Pedro Giffuni <pfg@FreeBSD.org> To: Bruce Evans <brde@optusnet.com.au>, araujo@freebsd.org Cc: John Baldwin <jhb@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r298247 - head/sbin/fdisk_pc98 Message-ID: <c53f4273-d84a-0321-5ba0-d8f99ac6366a@FreeBSD.org> In-Reply-To: <20160420115844.Y967@besplex.bde.org> References: <201604190446.u3J4kD9G050780@repo.freebsd.org> <4114217.PtcV9LDMal@ralph.baldwin.cx> <CAOfEmZhofgYySTtMKAXo-Qkm8BRaOP8kzEfchdtPTH3T-0W-qQ@mail.gmail.com> <20160420115844.Y967@besplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 04/19/16 22:06, Bruce Evans wrote: > On Wed, 20 Apr 2016, Marcelo Araujo wrote: > >> 2016-04-20 0:16 GMT+08:00 John Baldwin <jhb@freebsd.org>: >> >>> On Tuesday, April 19, 2016 04:46:13 AM Marcelo Araujo wrote: >>>> Author: araujo >>>> Date: Tue Apr 19 04:46:13 2016 >>>> New Revision: 298247 >>>> URL: https://svnweb.freebsd.org/changeset/base/298247 >>>> >>>> Log: >>>> Remove redundant parenthesis. >>>> >>>> Submitted by: pfg >>>> MFC after: 2 weeks. > > I don't realling like churnging to the nonstandard nitems(). Use > of the nonstandard <sys/param.h> is bad enough. > TBH, I am not joyful about them either, however it does improve readability so it makes sense only in the cases where sys/param.h is already in use. One of the things I dislike is that most of the macros are in lowercase. Lowercase would make sense if these were inline functions but inline functions have disadvantages for these use cases. OTOH, if they were uppercase, they would not be very easy on the eyes, which is the current advantage. No way out of it :(. Pedro.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?c53f4273-d84a-0321-5ba0-d8f99ac6366a>