Re: [HACKERS] Re: include-file cleanup

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: [HACKERS] Re: include-file cleanup
Date: 1999-07-17 16:17:01
Message-ID: 19919.932228221@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> writes:
> Well, the script just does the compile with and without the #include.

Oh ... I was assuming you had built something that actually went through
and gathered up a list of the symbols mentioned in each file.

I fear we are going to be putting back missing includes for a while to
come; in particular, I'll bet that MULTIBYTE and possibly USE_LOCALE are
now broken, unless you ran the script with those features enabled.
There are going to be a few more problems with platform-specific code
like the one I found in pqcomm.c, too.

What you did is a good hack as a one-shot cleanup, but I can't see
wanting to repeat it in future, not even as seldom as every year or two,
unless we build a much more reliable tool for the job.

regards, tom lane

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Hub.Org News Admin 1999-07-17 16:21:28
Previous Message Bruce Momjian 1999-07-17 15:38:43 Re: [HACKERS] Re: include-file cleanup