| 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: | Whole Thread | Raw Message | 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 |