> This recent patch is great but causes a small problem. It mixes designated > and non-designated initializers, specifically in `PG_MODULE_MAGIC_DATA(0)`.
Ugh. I felt a bit itchy about that, but my compiler wasn't complaining...
That's because this is valid in C99/C11; it's just not valid in C++. That said, I think it's confusing and error-prone.
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?
