Re: further #include cleanup (IWYU)

On 27.01.25 15:10, Matthias van de Meent wrote: On Wed, 6 Nov 2024 at 11:50, Peter Eisentraut wrote: Seeing no further comments (or any easy alternatives), I have committed this last patch as is. I just noticed that the #define for MaxArraySize in utils/array.h uses MaxAllocSize without incl

Re: further #include cleanup (IWYU)

On Wed, 6 Nov 2024 at 11:50, Peter Eisentraut wrote: > > Seeing no further comments (or any easy alternatives), I have committed > this last patch as is. I just noticed that the #define for MaxArraySize in utils/array.h uses MaxAllocSize without including the utils/memutils.h header. Is that on p

Re: further #include cleanup (IWYU)

Seeing no further comments (or any easy alternatives), I have committed this last patch as is. On 28.10.24 10:45, Peter Eisentraut wrote: On 20.10.24 11:53, Alvaro Herrera wrote: On 2024-Oct-20, Peter Eisentraut wrote: diff --git a/src/bin/pg_dump/pg_backup_utils.c b/src/bin/pg_dump/ pg_back

Re: further #include cleanup (IWYU)

On 20.10.24 11:53, Alvaro Herrera wrote: On 2024-Oct-20, Peter Eisentraut wrote: diff --git a/src/bin/pg_dump/pg_backup_utils.c b/src/bin/pg_dump/pg_backup_utils.c index a0045cf5e58..80715979a1a 100644 --- a/src/bin/pg_dump/pg_backup_utils.c +++ b/src/bin/pg_dump/pg_backup_utils.c @@ -13,7 +13,

Re: further #include cleanup (IWYU)

On 20.10.24 11:37, Alvaro Herrera wrote: On 2024-Oct-20, Peter Eisentraut wrote: diff --git a/contrib/tablefunc/tablefunc.h b/contrib/tablefunc/tablefunc.h index 2009382ce7d..b78030044b5 100644 --- a/contrib/tablefunc/tablefunc.h +++ b/contrib/tablefunc/tablefunc.h @@ -34,6 +34,4 @@ #ifndef T

Re: further #include cleanup (IWYU)

On 2024-Oct-20, Peter Eisentraut wrote: > diff --git a/src/bin/pg_dump/pg_backup_utils.c > b/src/bin/pg_dump/pg_backup_utils.c > index a0045cf5e58..80715979a1a 100644 > --- a/src/bin/pg_dump/pg_backup_utils.c > +++ b/src/bin/pg_dump/pg_backup_utils.c > @@ -13,7 +13,9 @@ > */ > #include "postgr

Re: further #include cleanup (IWYU)

On 2024-Oct-20, Peter Eisentraut wrote: > diff --git a/contrib/tablefunc/tablefunc.h b/contrib/tablefunc/tablefunc.h > index 2009382ce7d..b78030044b5 100644 > --- a/contrib/tablefunc/tablefunc.h > +++ b/contrib/tablefunc/tablefunc.h > @@ -34,6 +34,4 @@ > #ifndef TABLEFUNC_H > #define TABLEFUNC_H

further #include cleanup (IWYU)

This is a continuation of [0] ("backend *.c #include cleanup (IWYU)"), which removed a bunch of unneeded #include's, found by the include-what-you-use (IWYU) tool. I went through the rest of the code and did similar cleanups. The patches are organized by code area, but they are otherwise not