Re: Add new for_each macros for iterating over a List that do not require ListCell pointer

From: Jelte Fennema <postgres(at)jeltef(dot)nl>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Add new for_each macros for iterating over a List that do not require ListCell pointer
Date: 2023-10-24 16:58:04
Message-ID: CAGECzQQG7VX3LU4ccwygSeoVSM+UdSr+O8Mt40f0WEuoWV+ohg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, 24 Oct 2023 at 18:47, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
> BTW after applying your patches, initdb began failing with the following
> for me:
>
> TRAP: failed Assert("n >= 0 && n < list->length"), File: "list.c", Line: 770, PID: 902807

Oh oops... That was an off by one error in the modified
foreach_delete_current implementation.
Attached is a fixed version.

Attachment Content-Type Size
v2-0002-Use-new-for_each_xyz-macros-in-a-few-places.patch application/octet-stream 7.3 KB
v2-0001-Add-macros-for-looping-through-a-list-without-nee.patch application/octet-stream 4.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nathan Bossart 2023-10-24 17:04:51 Re: SLRU optimization - configurable buffer pool and partitioning the SLRU lock
Previous Message Mark Wong 2023-10-24 16:53:31 Re: LLVM 16 (opaque pointers)