Re: Fixed a typo in comment in compress_lz4.c

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Fixed a typo in comment in compress_lz4.c
Date: 2025-10-13 22:08:43
Message-ID: 829063.1760393323@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
>> On 13 Oct 2025, at 23:25, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
>> Fixed a typo: "iff" -> "if" that I found while reviewing the other patch.

> iff is shorthand for "if and only if", so it is correct here. That being said,
> it's also the cause of many typofix suggstions so it's clearly not widely known
> anymore and we try to avoid it new code/comments.

I disagree that it's not widely known. OneLook Dictionary Search [1]
defines it correctly as "Usually means: If and only if; equivalence"
and reports finding it in 35 of their underlying dictionaries,
including all the usual suspects. (Some of the 35 know the military
term "IFF: Identification Friend or Foe" rather than what we're
after, but still that's plenty of hits.)

I wonder whether adding it to our glossary would be helpful.

regards, tom lane

[1] https://www.onelook.com/?w=iff

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2025-10-13 22:11:38 Re: Clarification on Role Access Rights to Table Indexes
Previous Message Tom Lane 2025-10-13 21:59:37 Re: [PATCH] Fix incorrect fprintf usage in log_error FRONTEND path