Alvaro Herrera <alvhe...@commandprompt.com> writes:
> Bruce Momjian wrote:
>> Tom Lane wrote:
>>> I propose changing spi.h to follow the same include-only-what-you-must
>>> rule as every other backend header file.  Thoughts?
>> 
>> I don't think we ever cleaned out spi.h in the past because we were
>> worried about 3rd party code using it (I am fine with a cleanup).

> I've wondered about spi.h lately too while looking at header cleanup,
> and I agree with the proposed solution.  The worst that can happen is
> that somebody needs to add extra includes in their programs in order for
> them to compile with 8.4.  We do enough other changes that this one is
> really minor.  Better late than never anyway.

Okay, I'll do a trial patch and we can see exactly how much has to be
added (at least among core and contrib) before deciding for sure.

                        regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to