Re: include-file cleanup
| От | Tom Lane |
|---|---|
| Тема | Re: include-file cleanup |
| Дата | |
| Msg-id | 19088.932187118@sss.pgh.pa.us обсуждение исходный текст |
| Ответ на | Re: include-file cleanup (Bruce Momjian <maillist@candle.pha.pa.us>) |
| Ответы |
Re: include-file cleanup
|
| Список | pgsql-hackers |
> I have reviewed and replaced config.h in all files it appeared in in
> 6.5, where postgres.h or c.h were not already included. I have also
> removed config.h from the cleaning script, just as postgres.h was never
> removed.
OK, that sounds good.
The thing that bothers me is why config.h got removed from these
port files in the first place. The compiler warning I got (because
I use gcc -Wmissing-prototypes) was that "random" and "srandom"
were defined without having been declared in any include file.
Now config.h provides prototypes for those functions --- inside
#ifdefs of course, but they are there. Your script should have
noticed that the name "random" mentioned in config.h was also
mentioned in port/random.c, and therefore not removed the include
of config.h from random.c. Why did it not make the connection?
regards, tom lane
В списке pgsql-hackers по дате отправления: