Horton

Re: anonymous unions (C11)

Peter Eisentraut <peter@eisentraut.org>

View thread View on postgr.es
From: Peter Eisentraut <peter@eisentraut.org>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-10-03T07:01:26Z
Lists: pgsql-hackers 
On 23.09.25 16:17, Nathan Bossart wrote:
> On Tue, Sep 23, 2025 at 11:38:22AM +0200, Peter Eisentraut wrote:
>> That said, I did go overboard here and converted all the struct/union
>> combinations I could find, but I'm not necessarily proposing to apply
>> all of them.  I'm proposing patches 0001 through 0004, which are
>> relatively simple or in areas that have already changed a few times
>> recently (so backpatching would not be trivial anyway), and/or they
>> are somewhat close to my heart because they originally motivated this
>> work a long time ago.  But if someone finds among the other patches
>> one that they particularly like, we could add that one as well.
> 
> I would have used this in the DSM registry if it was available.  Patch
> attached.

This looks good to me, and also mostly harmless in terms of backpatching.

Commits

  1. Make some use of anonymous unions [reloptions]

  2. Make some use of anonymous unions [libpq-oauth]

  3. Make some use of anonymous unions [DSM registry].

  4. Make some use of anonymous unions [pg_locale_t]

  5. Make some use of anonymous unions [pgcrypto]

  6. Make some use of anonymous unions [plpython]

  7. Make some use of anonymous unions [reorderbuffer xact_time]