| From: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> |
|---|---|
| To: | Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Cc: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
| Subject: | backend/nodes cleanup: Move loop variables definitions into for statement |
| Date: | 2025-11-12 08:00:19 |
| Message-ID: | CAEoWx2nP12qwAaiJutbn1Kw50atN6FbMJNQ4bh4+fP_Ay_u7Eg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Hackers,
While working on the other patch [1] that touched tbm_add_tuples() under
src/backend/nodes, I moved a loop variable into for statement by the way as
I knew Peter Eisentraut had done some efforts about that, see [2].
However, Peter removed that change from the patch and he said that should
belong to a separate cleanup patch. So I am following up and moving loop
variables into for statements wherever possible under src/backend/nodes.
[1]
https://www.postgresql.org/message-id/flat/CAEoWx2m2E0xE8Kvbkv31ULh_E+5zph-WA_bEdv3UR9CLhw+3vg(at)mail(dot)gmail(dot)com
[2]
https://git.postgresql.org/cgit/postgresql.git/commit/?id=03fbb0814c5015ab79e670ab97bb6a3349269e4b
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
| Attachment | Content-Type | Size |
|---|---|---|
| v1-0001-backend-nodes-cleanup-Move-loop-variables-definit.patch | application/octet-stream | 8.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nazir Bilal Yavuz | 2025-11-12 08:04:28 | Re: Trying out read streams in pgvector (an extension) |
| Previous Message | Michael Paquier | 2025-11-12 07:58:43 | Re: [Patch] Windows relation extension failure at 2GB and 4GB |