Hi,

On Tue, Mar 24, 2026 at 05:16:11PM +0100, Álvaro Herrera wrote:
> Hello,
> 
> On 2026-Mar-23, Bertrand Drouvot wrote:
> 
> > 2/ Not directly linked to your patches
> > 
> > It looks like that aio_funcs.c does not need lock.h (reported by 
> > include-what-you-use).
> > If we remove its direct include, it's still indirectly included through 
> > proc.h
> > though. But I think that removing its direct include makes sense as it's not
> > needed at all.
> 
> I'm not opposed to somebody else making this change, if they want, but I
> think there's little practical impact.

yeah, not sure it deserves its own commit... I was just mentioning here in case
you want to change that in passing while working on lock.h includes.

Regards,

-- 
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com


Reply via email to