| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
| Cc: | Rafia Sabih <rafia(dot)pghackers(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Minor spelling fix in memnodes.h |
| Date: | 2025-10-17 14:37:22 |
| Message-ID: | 1678307.1760711842@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
David Rowley <dgrowleyml(at)gmail(dot)com> writes:
> On Fri, 17 Oct 2025 at 20:39, Rafia Sabih <rafia(dot)pghackers(at)gmail(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2025-10-17 14:49:28 | Re: Getting the SQLSTATE after a failed connection |
| Previous Message | Fujii Masao | 2025-10-17 14:28:17 | Re: Fix lag columns in pg_stat_replication not advancing when replay LSN stalls |