Horton

Re: Minor spelling fix in memnodes.h

Tom Lane <tgl@sss.pgh.pa.us>

View thread
From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <dgrowleyml@gmail.com>
Cc: Rafia Sabih <rafia.pghackers@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-10-17T14:37:22Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Englishify comment wording

David Rowley <dgrowleyml@gmail.com> writes:
> On Fri, 17 Oct 2025 at 20:39, Rafia Sabih <rafia.pghackers@gmail.com> wrote:
>> Came across this spelling error alloced -> allocated.
>> Patch for the fix is attached.

> I suspect it is written as Tom intended it. The callback function in
> MemoryContextMethods is called 'alloc' so I guess "alloced" is the
> function name as a verb.

Yeah, it's not a typo but an intentional abbreviation.  Having said
that, I don't have any objection to changing it, especially since
there's now a use of "allocated" just two lines away.

I suspect that I shortened it to "alloced" in service of keeping the
comment on one line ... but it looks like it'd still fit if we added
the additional two characters.

			regards, tom lane