From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-hackers@lists.postgresql.org
Cc: Тарасов Георгий Витальевич <Tarasov-G@gaz-is.ru>
Subject: Re: compiling PL/pgSQL plugin with C++
Date: Thu, 30 May 2019 17:46:28 -0400
Message-ID: <74481.1559252788@sss.pgh.pa.us> (raw)
In-Reply-To: <51937.1559231664@sss.pgh.pa.us>
References: <b517ec3918d645eb950505eac8dd434e@gaz-is.ru>
<51937.1559231664@sss.pgh.pa.us>
I wrote:
> I propose that we change src/tools/pginclude/cpluspluscheck so that
> it searches basically everywhere:
> -for f in `find src/include src/interfaces/libpq/libpq-fe.h src/interfaces/libpq/libpq-events.h -name '*.h' -print | \
> +for f in `find src contrib -name '*.h' -print | \
After further experimentation with that, it seems like we'll have
to continue to exclude src/bin/pg_dump/*.h from the C++ check.
pg_dump uses "public" and "namespace" as field names in various
structs, both of which are C++ keywords. Changing these names
would be quite invasive, and at least in the short run I see no
payoff for doing so.
ecpg/preproc/type.h is also using "new" as a field name, but it
looks like there are few enough references that renaming that
field isn't unreasonable.
There are various other minor issues, but they generally look
fixable with little consequence.
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-general@postgresql.org
Cc: tgl@sss.pgh.pa.us, pgsql-hackers@lists.postgresql.org, Tarasov-G@gaz-is.ru
Subject: Re: compiling PL/pgSQL plugin with C++
In-Reply-To: <74481.1559252788@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox