| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Thomas Lockhart <lockhart(at)fourpalms(dot)org> |
| Cc: | pgsql-hackers(at)postgreSQL(dot)org |
| Subject: | #warning possibly dangerous? |
| Date: | 2002-04-29 17:26:23 |
| Message-ID: | 29883.1020101183@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
In utils/int8.h we currently have
/* this should be set in pg_config.h, but just in case it wasn't: */
#ifndef INT64_FORMAT
#warning "Broken pg_config.h should have defined INT64_FORMAT"
#define INT64_FORMAT "%ld"
#endif
I would like to remove this. The #warning command is not standard C.
I get a warning about it from HP's cc, and it may produce hard errors
on other non-gcc compilers with even less forgiving preprocessors.
We could just take out that one line --- but we are not in the habit of
backstopping configure/pg_config.h for any other settings, so I don't
see the point of doing it for INT64_FORMAT. I'd like to take out all
five lines.
Objections?
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Hannu Krosing | 2002-04-29 17:39:25 | Re: Vote totals for SET in aborted transaction |
| Previous Message | Doug McNaught | 2002-04-29 17:14:43 | Re: Vote totals for SET in aborted transaction |