Horton

Re: Add Postgres module info

Tom Lane <tgl@sss.pgh.pa.us>

View thread View on postgr.es
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Yurii Rashkovskii <yrashk@omnigres.com>
Cc: Andrei Lepikhov <lepihov@gmail.com>, Euler Taveira <euler@eulerto.com>, Robert Haas <robertmhaas@gmail.com>, Michael Paquier <michael@paquier.xyz>, Alexander Korotkov <aekorotkov@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-03-27T03:17:17Z
Lists: pgsql-hackers 
Yurii Rashkovskii <yrashk@omnigres.com> writes:
>> Can you propose a specific change to clean it up?  I wanted to write
>> just "PG_MODULE_MAGIC_DATA()", but I'm not sure that's valid C either.

> I was thinking about passing `.name = NULL, .version = NULL` instead of
> `0`—do you have any reservations about this?

If we're going that way, I'd minimize it to just ".name = NULL".

			regards, tom lane

Commits

  1. Avoid mixing designated and non-designated field initializers.

  2. Use PG_MODULE_MAGIC_EXT in our installable shared libraries.

  3. Introduce PG_MODULE_MAGIC_EXT macro.