| From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Brian P Millett <bpm(at)ec-group(dot)com>, postgres <hackers(at)postgreSQL(dot)org> |
| Subject: | Re: [HACKERS] PATCH for pgconnection.h |
| Date: | 1999-07-09 03:50:28 |
| Message-ID: | 199907090350.XAA07225@candle.pha.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:
> >>
> >> + #ifdef __sun__
> >> + #ifndef __GNUC__
> >> + using namespace std;
> >> + #endif
> >> + #endif
Yes, never applied.
> The above is really, really ugly, not to say broken, because neither
> being on a Sun nor using gcc have anything to do with whether your
> compiler handles namespaces. The problem we are looking at here is that
> the C++ standard is a moving target, and some people have compilers that
> are newer than others.
>
> I think the proper solution is to add a configure-time test to see
> whether a namespace declaration is needed. We could use configure to
> see whether we need ".h" on the end of C++ include file references, too.
> (That's another thing that's going to be site-dependent for a while to
> come.)
I smell TODO list:
* Add configure test to check for C++ need for *.h and namespaces
Added.
--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 1999-07-09 03:56:36 | Re: [HACKERS] PATCH for pgconnection.h |
| Previous Message | Vadim Mikheev | 1999-07-09 03:34:00 | Re: [HACKERS] Delaying insertion of default values |