Re: [HACKERS] PATCH for pgconnection.h

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

In response to

Responses

Browse pgsql-hackers by date

  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