Re: Make copyObject work in C++

Jelte Fennema <postgres@jeltef.nl>

View thread

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>

Date: 2026-01-25T12:42:51Z

Lists: pgsql-hackers

Attachments

v7-0001-Support-building-C-extensions-with-MSVC.patch (text/x-patch) patch v7-0001
v7-0002-Support-using-copyObject-in-C.patch (text/x-patch) patch v7-0002
v7-0003-Use-pg_exprtype-instead-of-typeof.patch (text/x-patch) patch v7-0003
v7-0004-Support-using-StaticAssert-macros-in-C.patch (text/x-patch) patch v7-0004
v7-0005-Support-using-list_make-macros-in-C.patch (text/x-patch) patch v7-0005

On Tue Jan 20, 2026 at 5:28 PM CET, Peter Eisentraut wrote:
> I have split your first patch further.  For a start, I left out the 
> PG_MODULE_MAGIC*-related changes and disabled the module under MSVC. 
> This has been committed.  I plan to let the buildfarm run with it for a 
> day or two and then add in the basic MSVC support.

To hopefully make your life a bit easier. Here's a rebased version that
enables the MSVC support again, with an updated commit message.

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