Re: Typos in the code and README

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

>

In response to

Responses

Browse pgsql-hackers by date

  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