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>
Date: 2026-02-20T09:47:40Z
Lists: pgsql-hackers

Attachments

On Thu Feb 19, 2026 at 9:31 AM CET, Peter Eisentraut wrote:
> What I suggest we should do here is:
>
> 1. Add a configure test that checks if the C++ compiler supports typeof.
>
> 2. If not, #define typeof to the above expression.
>
> Then code can continue to use typeof unchanged.

Makes total sense, I didn't realise decltype and typeof were not quite
the same thing. Attached is an updated patchset that does that.

It also includes a patch that improves unconstify and unvolatize by
using StaticAssertVariableIsOfTypeMacro instead of a custom version of
that assertion. (Like I said, in a future patch I intend to make
StaticAssertVariableIsOfTypeMacro work in C++ as well, but this seemed
like a good improvement anyway)

Commits

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

  2. Disable some C++ warnings in MSVC

  3. meson: Make room for C++-only warning flags for MSVC

  4. Make fixed-length list building macros work in C++

  5. Make unconstify and unvolatize use StaticAssertVariableIsOfTypeMacro

  6. Use typeof everywhere instead of compiler specific spellings

  7. Test List macros in C++ extensions

  8. Test most StaticAssert macros in C++ extensions

  9. Revert "Change copyObject() to use typeof_unqual"

  10. Fix for C++ compatibility

  11. tests: Add a test C++ extension module