Re: Make copyObject work in C++
Peter Eisentraut <peter@eisentraut.org>
From: Peter Eisentraut <peter@eisentraut.org>
To: Jelte Fennema-Nio <postgres@jeltef.nl>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
Thomas Munro <thomas.munro@gmail.com>, Andres Freund <andres@anarazel.de>
Date: 2026-03-27T11:23:24Z
Lists: pgsql-hackers
Attachments
- v11-0001-meson-Make-room-for-C-only-warning-flags-for-MSV.patch (text/plain) patch v11-0001
- v11-0002-Disable-some-C-warnings-in-MSVC.patch (text/plain) patch v11-0002
- v11-0003-Enable-test_cplusplusext-with-MSVC.patch (text/plain) patch v11-0003
On 23.03.26 10:58, Jelte Fennema-Nio wrote: > On Mon, 23 Mar 2026 at 10:27, Peter Eisentraut <peter@eisentraut.org> wrote: >> I think we should commit the pg_list.h changes, since the C-style >> compound literals are not a C++ feature at all, and so without this MSVC >> would never get supported. (Or you couldn't use PostgreSQL lists, which >> would be very limiting.) > > Sounds good to me. This has been committed. >> The other changes deal with designated initializers and flexible array >> members. These are not a blocker, since extension authors could deal >> with them themselves by adding appropriate compiler options or similar. > > I think we should add these flags to CXXFLAGS for MSVC by default, > similar to how we add -std=gnu++11/-std=c++11 for other compilers. We > can then document on the C++ extension docs page, that MSVC compilers > require C++20 support. 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.
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