Re: pg_checksums: Reorder headers in alphabetical order
От | Nathan Bossart |
---|---|
Тема | Re: pg_checksums: Reorder headers in alphabetical order |
Дата | |
Msg-id | Zu7HBDK7_6bp2qgs@nathan обсуждение исходный текст |
Ответ на | Re: pg_checksums: Reorder headers in alphabetical order (Fujii Masao <masao.fujii@oss.nttdata.com>) |
Список | pgsql-hackers |
On Sat, Sep 21, 2024 at 02:48:32PM +0900, Fujii Masao wrote: > On 2024/09/21 12:09, Tom Lane wrote: >> Fujii Masao <masao.fujii@oss.nttdata.com> writes: >> > I don´t have any objections to this commit, but I´d like to confirm >> > whether we really want to proactively reorder #include directives, >> > even for standard C library headers. >> >> I'm hesitant to do that. We can afford to insist that our own header >> files be inclusion-order-independent, because we have the ability to >> fix any problems that might arise. We have no ability to do something >> about it if the system headers on $random_platform have inclusion >> order dependencies. (In fact, I'm fairly sure there are already >> places in plperl and plpython where we know we have to be careful >> about inclusion order around those languages' headers.) >> >> So I would tread pretty carefully around making changes of this >> type, especially in long-established code. I have no reason to >> think that the committed patch will cause any problems, but >> I do think it's mostly asking for trouble. > > Sounds reasonable to me. Oh, sorry. I thought it was project policy to keep these alphabetized, and I was unaware of the past problems with system header ordering. I'll keep this in mind in the future. -- nathan
В списке pgsql-hackers по дате отправления: