Re: Fix comment typo in /src/backend/command/cluster.c - Mailing list pgsql-hackers

From David Fetter
Subject Re: Fix comment typo in /src/backend/command/cluster.c
Date
Msg-id 20140128065816.GB29525@fetter.org
Whole thread Raw
In response to Re: Fix comment typo in /src/backend/command/cluster.c  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: Fix comment typo in /src/backend/command/cluster.c  (Heikki Linnakangas <hlinnakangas@vmware.com>)
List pgsql-hackers
On Tue, Jan 28, 2014 at 02:51:22PM +0900, Michael Paquier wrote:
> On Tue, Jan 28, 2014 at 2:29 PM, David Fetter <david@fetter.org> wrote:
> > On Tue, Jan 28, 2014 at 04:48:35PM +1300, Gavin Flower wrote:
> >> On 28/01/14 16:33, Andrew Dunstan wrote:
> >> >On 01/27/2014 10:24 PM, Sawada Masahiko wrote:
> >> >>Hi all,
> >> >>
> >> >>Attached patch fixes the typo which is in
> >> >>"src/backend/command/cluster.c".
> >> >
> >> >Are you sure that's a typo? "iff" is usually short hand for "if
> >> >and only if".
> >> >
> >> >cheers
> >> >
> >> >andrew
> >> >
> >> >
> >> Certainly, that is how I would interpret it.
> >>
> >> I came across that abbreviation in a first years Maths course
> >> "Principles of Mathematics" in 1968 at the University of Auckland..
> >
> > By my rough count (ack -l '\biff\b' |wc -l), it's used to mean
> > equivalence 81 times in the source tree.  Should we have a glossary of
> > such terms?
> And what about directly replacing those expressions in the comments of
> the code with some more understandable language? This would be more
> suited for non-native English speakers than maintaining a glossary
> that you can surely find here and there after some googling.

I'm interested to find 29 instances of "if and only if" in the source,
which should be the same thing.

Please find attached a mechanically done patch which expands the
remaining instances of "iff" to the longer form, all of which are in
comments.  The patched source passes make -j8, but I have not tested
it further.

Cheers,
David.
--
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

Attachment

pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: [COMMITTERS] pgsql: Keep pg_stat_statements' query texts in a file, not in shared me
Next
From: Heikki Linnakangas
Date:
Subject: Re: git-review: linking commits to review discussion in git