Horton

Re: further #include cleanup (IWYU)

Matthias van de Meent <boekewurm+postgres@gmail.com>

View thread View on postgr.es
From: Matthias van de Meent <boekewurm+postgres@gmail.com>
To: Peter Eisentraut <peter@eisentraut.org>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-01-27T14:10:18Z
Lists: pgsql-hackers 
On Wed, 6 Nov 2024 at 11:50, Peter Eisentraut <peter@eisentraut.org> 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
purpose and is the user expected to include both headers, or should
utils/memutils.h be included in utils/array.h?

Kind regards,

Matthias van de Meent
Neon (https://neon.tech)

Commits

  1. Remove unused #include's from bin .c files

  2. Remove unused #include's from contrib, pl, test .c files

  3. Remove unused #include's from backend .c files

  4. Replace most usages of ntoh[ls] and hton[sl] with pg_bswap.h.