Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rename foreach_ macros to foreach_declared_ macros #7700

Open
wants to merge 1 commit into
base: release-13.0
Choose a base branch
from
Open

Rename foreach_ macros to foreach_declared_ macros #7700

wants to merge 1 commit into from
+802 −802

Commits on Oct 6, 2024

  1. Rename foreach_ macros to foreach_declared_ macros 

    PG17 added foreach_ptr, foreach_int and foreach_oid macros
Relevant PG commit
14dd0f27d7cd56ffae9ecdbe324965073d01a9ff
postgres/postgres@14dd0f2

We already have these macros, but they are different with the
PG17 ones because our macros take a DECLARED variable, whereas
the PG16 macros declare a locally-scoped loop variable themselves.

Hence I am renaming our macros to foreach_declared_
    @naisila
    naisila committed Oct 6, 2024
    Configuration menu
    Copy the full SHA
    be91508 View commit details
    Browse the repository at this point in the history