Re: Array initialisation notation in syscache.c

On Thu, Sep 21, 2023 at 8:19 PM Peter Eisentraut <peter@eisentraut.org> wrote:
> On 31.03.23 04:16, Thomas Munro wrote:
> >  From the light relief department, here is some more variadic macrology:
> >
> > -       tp = SearchSysCache1(TSPARSEROID, ObjectIdGetDatum(prsId));
> > +       tp = SearchSysCache(TSPARSEROID, ObjectIdGetDatum(prsId));
>
> I'm worried that if we are removing the variants with the explicit
> numbers, it will make it difficult for extensions to maintain
> compatibility with previous PG major versions.  They would probably have
> to copy much of your syscache.h changes into their own code.  Seems messy.

I suppose we could also supply a set of macros with the numbers that
map straight onto the numberless ones, with a note that they will be
deleted after N releases.  But maybe not worth the hassle for such a
tiny improvement in core code readability.  I will withdraw this
entry.  Thanks.