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

2025-06-30 Thread Peter Eisentraut
On 16.06.25 16:20, Aleksander Alekseev wrote: The removals of bother me a bit; I believe those used to be necessary on some platforms. Maybe nowadays everybody is close enough to POSIX that you can safely extrapolate from what clangd says on your own machine, but it's not zero-risk. Sounds go

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

2025-06-16 Thread Aleksander Alekseev
Hi Tom, > The removals of bother me a bit; I believe those used > to be necessary on some platforms. Maybe nowadays everybody is > close enough to POSIX that you can safely extrapolate from what > clangd says on your own machine, but it's not zero-risk. Sounds good to me, let's keep . -- Best

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

2025-06-16 Thread Tom Lane
Aleksander Alekseev writes: > clangd indicates that certain #include's are redundant. Removing them > will speed up the build process a bit. The removals of bother me a bit; I believe those used to be necessary on some platforms. Maybe nowadays everybody is close enough to POSIX that you can sa

[PATCH] Remove unused #include's in src/backend/utils/adt/*

2025-06-16 Thread Aleksander Alekseev
Hi, clangd indicates that certain #include's are redundant. Removing them will speed up the build process a bit. -- Best regards, Aleksander Alekseev v1-0001-Remove-unused-include-s-in-src-backend-utils-adt.patch Description: Binary data