Re: Make copyObject work in C++

Jelte Fennema <postgres@jeltef.nl>

From: "Jelte Fennema-Nio" <postgres@jeltef.nl>

To: "Peter Eisentraut" <peter@eisentraut.org>

Cc: "PostgreSQL Hackers" <pgsql-hackers@lists.postgresql.org>, "Thomas Munro" <thomas.munro@gmail.com>, "Andres Freund" <andres@anarazel.de>

Date: 2026-03-29T22:53:48Z

Lists: pgsql-hackers

Attachments

v12-0001-meson-Make-room-for-C-only-warning-flags-for-MSV.patch (text/x-patch) patch v12-0001
v12-0002-Disable-some-C-warnings-in-MSVC.patch (text/x-patch) patch v12-0002
v12-0003-Add-support-for-C-extensions-with-MSVC.patch (text/x-patch) patch v12-0003
v12-0004-meson-Consider-an-insufficient-C-compiler-a-warn.patch (text/x-patch) patch v12-0004

On Fri, 27 Mar 2026 at 12:23, Peter Eisentraut <peter@eisentraut.org> wrote:
> Here is another tidied up patch set for this.  I didn't go quite as far
> as enabling C++20 by default in meson.build, this would just take more
> time to work out and test all the different combinations, but I added
> the flag to the Cirrus CI task, since there we know what compiler we have.

I think 0001 and 0002 are good.

0003 seems awkward though. Attached is an approach that I think is
better: It actually checks for the required featureset and adds the
necessary flags to the compiler.

I also added a small patch in 0004 to align configure and meson
behaviour when no sufficiently modern compiler is found.

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Enable test_cplusplusext with MSVC
- c05ad248f99c 19 (unreleased) landed
Disable some C++ warnings in MSVC
- 82a7cbea747c 19 (unreleased) landed
meson: Make room for C++-only warning flags for MSVC
- 4c83f1253593 19 (unreleased) landed
Make fixed-length list building macros work in C++
- f8e7ca328510 19 (unreleased) landed
Make unconstify and unvolatize use StaticAssertVariableIsOfTypeMacro
- 258248d0bdbf 19 (unreleased) landed
Use typeof everywhere instead of compiler specific spellings
- e2308350c9b7 19 (unreleased) landed
Test List macros in C++ extensions
- 3d28ecb5ac76 19 (unreleased) landed
Test most StaticAssert macros in C++ extensions
- 451650eaacd5 19 (unreleased) landed
Revert "Change copyObject() to use typeof_unqual"
- 0af05b5dbb42 19 (unreleased) landed
Fix for C++ compatibility
- b4555cb070f1 19 (unreleased) landed
tests: Add a test C++ extension module
- 476b35d4e311 19 (unreleased) landed