[PATCH] Make spelling consistent in waiteventset.c
Hi,
The four backend-specific paths in WaitEventSetWaitBlock()
(epoll, kqueue, poll, and Windows) had identical-purpose comments
with inconsistent spellings.
Two comments used "writable" while the other two used "writeable".
This patch standardizes the spelling to "writable" to match the
surrounding text in the file.
The WL_SOCKET_WRITEABLE macro is a public identifier and is left
unchanged.
Patch attached.
Attachments:
0001-Make-spelling-consistent-in-waiteventset.c.patchtext/x-patch; charset=US-ASCII; name=0001-Make-spelling-consistent-in-waiteventset.c.patchDownload+2-3
On Wed, May 6, 2026 at 03:17:52PM +0530, Postgres Cybrosys wrote:
Hi,
The four backend-specific paths in WaitEventSetWaitBlock()
(epoll, kqueue, poll, and Windows) had identical-purpose comments
with inconsistent spellings.Two comments used "writable" while the other two used "writeable".
This patch standardizes the spelling to "writable" to match the
surrounding text in the file.The WL_SOCKET_WRITEABLE macro is a public identifier and is left
unchanged.Patch attached.
I went to apply this patch and then got stuck when filling out the
Author field, since the email name doesn't look like a person's name,
and I am uncomfortable committing something without a person's name and
I don't remember ever doing this. To be more concrete, without a name
there is no way to reference the original author in the future --- a
company name doesn't allow that.
I am willing to apply this patch with no author if that is what people
prefer, or we can just leave the C comment typo unfixed.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EDB https://enterprisedb.com
Do not let urgent matters crowd out time for investment in the future.