[PATCH] Fix typos in pqsignal.c comment
Hackorum builds and tests every patch posted to the lists, not only commitfest submissions. This is Hackorum's own CI rather than the PostgreSQL project's, and it is still under testing - please report anything that looks wrong.
You can run a PostgreSQL built from this patch straight from Docker, with no checkout and no build:
docker run --rm -p 5432:5432 ghcr.io/hackorum-dev/postgres-patch:t139781psql -h localhost -U postgresBuilt from patchset v1 (message #1), July 26, 2026 at 11:53 PM.
Every patchset is also pushed to a branch of our PostgreSQL fork, so you can check out the same tree CI built. Without a PostgreSQL checkout:
git clone --branch t139781_1 https://github.com/hackorum-dev/postgres.gitIn a checkout you already have, add the fork once:
git remote add hackorum https://github.com/hackorum-dev/postgres.gitthen, for this patchset and every later one:
git fetch hackorum t139781_1 && git checkout t139781_1Patchset v1 (message #1) is on t139781_1
Hi hackers,
The attached patch fixes two typos in a comment in src/port/pqsignal.c:
- "this though wrapper handler" -> "this through wrapper handler"
- "as it it needs to take" -> "as it needs to take"
Comment-only change; no behavior change.
Regards,
Ewan Young
Hi,
For tracking, I've added this to the open commitfest:
https://commitfest.postgresql.org/patch/6871/
Regards,
Ewan Young
Show quoted text
On Fri, May 29, 2026 at 2:01 PM kdbase <kdbase.hack@gmail.com> wrote:
Hi hackers,
The attached patch fixes two typos in a comment in src/port/pqsignal.c:
- "this though wrapper handler" -> "this through wrapper handler"
- "as it it needs to take" -> "as it needs to take"Comment-only change; no behavior change.
Regards,
Ewan Young
On Jun 10, 2026, at 15:18, Ewan Young <kdbase.hack@gmail.com> wrote:
Hi,
For tracking, I've added this to the open commitfest:
https://commitfest.postgresql.org/patch/6871/Regards,
Ewan YoungOn Fri, May 29, 2026 at 2:01 PM kdbase <kdbase.hack@gmail.com> wrote:
Hi hackers,
The attached patch fixes two typos in a comment in src/port/pqsignal.c:
- "this though wrapper handler" -> "this through wrapper handler"
- "as it it needs to take" -> "as it needs to take"Comment-only change; no behavior change.
Regards,
Ewan Young
I think this change will go into Michael's bag, so it doesn't need to be added to the CF.
FYI, Michael keeps collecting these kinds of tiny changes, such as typo fixes, and pushes them all together every couple of months.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
Thanks for letting me know — that's good to hear.
I'll withdraw the CommitFest entry then, and leave the patch here in
the thread for Michael to pick up whenever he next batches these
together.
Appreciate the pointer!
Best regards,
Ewan
Show quoted text
On Wed, Jun 10, 2026 at 3:28 PM Chao Li <li.evan.chao@gmail.com> wrote:
On Jun 10, 2026, at 15:18, Ewan Young <kdbase.hack@gmail.com> wrote:
Hi,
For tracking, I've added this to the open commitfest:
https://commitfest.postgresql.org/patch/6871/Regards,
Ewan YoungOn Fri, May 29, 2026 at 2:01 PM kdbase <kdbase.hack@gmail.com> wrote:
Hi hackers,
The attached patch fixes two typos in a comment in src/port/pqsignal.c:
- "this though wrapper handler" -> "this through wrapper handler"
- "as it it needs to take" -> "as it needs to take"Comment-only change; no behavior change.
Regards,
Ewan YoungI think this change will go into Michael's bag, so it doesn't need to be added to the CF.
FYI, Michael keeps collecting these kinds of tiny changes, such as typo fixes, and pushes them all together every couple of months.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
