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
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 |