Re: Make copyObject work in C++

Jelte Fennema <postgres@jeltef.nl>

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

To: "Peter Eisentraut" <peter@eisentraut.org>, "PostgreSQL Hackers" <pgsql-hackers@lists.postgresql.org>, "Thomas Munro" <thomas.munro@gmail.com>

Date: 2026-01-03T09:32:28Z

Lists: pgsql-hackers

Attachments

v4-0001-Make-copyObject-work-in-C.patch (text/x-patch) patch v4-0001
v4-0002-Use-pg_exprtype-instead-of-typeof.patch (text/x-patch) patch v4-0002
v4-0003-Revert-Replace-pg_restrict-by-standard-restrict.patch (text/x-patch) patch v4-0003
v4-0004-tests-Add-a-test-C-extension-module.patch (text/x-patch) patch v4-0004

On Tue Dec 16, 2025 at 1:28 PM CET, Peter Eisentraut wrote:
> I think it might be good to create a test extension written in C++, like 
> under src/test/modules/, and sprinkle it with various constructs like 
> copyObject() and static assertions, and whatever else we find that is 
> possibly problematic.

Attached is a patchset that does that. It required a few more fixes to
make the extension compile on MSVC too.

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