| From: | Alexander Law <exclusion(at)gmail(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | David Rowley <dgrowleyml(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
| Subject: | Re: Typos in the code and README |
| Date: | 2026-01-05 19:00:00 |
| Message-ID: | CAJ0YPFFWhJXs-e-=7iJz-FLp=b1dXfJA_qtrVAgto=bZmzD9zQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello Andres,
On Mon, Jan 5, 2026, 21:24 Andres Freund <andres(at)anarazel(dot)de> wrote:
>
> On 2026-01-01 10:00:00 +0200, Alexander Lakhin wrote:
> > bufhdr -> bufHdr
>
> I just ran into this one due to a rebase conflict. Obviously trivial to
> resolve. But what's the point of this change? If you wanted to fix the
> arguably wrong name, it'd have to be "buf" or "BufferDesc", as there's no
> variable named either bufhdr or bufHdr.
>
I thought that the initial spelling was meant to reference bufHdr variable
as if it was declared locally as we can see this name in other functions in
bufmgr.c (at 5d508736).
The point was to have no references to entities that you can't find in the
source tree.
> Having random stuff mixed into these changes is one of the reasons why I so
> dislike them.
>
I'm not sure how to arrange such changes to process them efficiently --
they could be split to multiple "Fix a typo" patches, of course.
By the way, I've just noticed that I fixed only one out of two typos in
"parititoned", so it's still spelled wrong: "partititoned". Sorry.
Best regards,
Alexander
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrey Borodin | 2026-01-05 19:10:53 | Re: REASSIGN OWNED BY alters objects in other database. |
| Previous Message | Andres Freund | 2026-01-05 18:58:51 | Re: Adding NetBSD and OpenBSD to Postgres CI |