On Thu, Mar 5, 2026 at 10:57 AM Japin Li <[email protected]> wrote:
> Just a quick note: I think `foreach_ptr` is more appropriate here than 
> `foreach`.

Agreed.

For homogeneous lists, it is better to use foreach_node so we get the
additional type-safety assertions in dev builds.  For heterogeneous
lists, we should use foreach_ptr.

Attached is an updated version of the patch that replaces foreach with
foreach_ptr or foreach_node accordingly.  Nothing else has changed.

I plan to do one more round of self-review on this.  Unless there are
any further thoughts or concerns, I'm hoping to commit this in a week
or two.  Please let me know if anyone spots anything else I might have
missed.

- Richard

Attachment: v5-0001-Convert-NOT-IN-sublinks-to-anti-joins-when-safe.patch
Description: Binary data

Reply via email to