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