Re: [PATCH] Remove unused #include's in src/backend/commands/*

From: Nathan Bossart <nathandbossart(at)gmail(dot)com>
To: Aleksander Alekseev <aleksander(at)tigerdata(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] Remove unused #include's in src/backend/commands/*
Date: 2025-10-08 16:50:34
Message-ID: aOaWWibezASiQV7L@nathan
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Oct 08, 2025 at 05:30:58PM +0300, Aleksander Alekseev wrote:
>> clangd indicates that certain #include's are redundant. Removing them
>> will speed up the build process a bit.
>
> OK, cfbot tells me that touching collationcmds.c was a mistake. Here
> is a corrected patch.

Seems reasonable to me. Most of these seem to be recent additions since
last year's round of IWYU commits. Any reason to limit this to
src/backend/commands? Why not tackle the whole tree at once?

--
nathan

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Álvaro Herrera 2025-10-08 16:56:58 Re: VACUUM (PARALLEL) option processing not using DefElem the way it was intended
Previous Message Nathan Bossart 2025-10-08 16:41:34 Re: plpython: Remove support for major version conflict detection