On Tue, 20 Jan 2026 at 08:49, Peter Eisentraut <[email protected]> wrote:
>
> On 05.12.25 17:38, Heikki Linnakangas wrote:
> > #if 0
> > Oid subtype = PG_GETARG_OID(3);
> > #endif
> >
> > is yet another option. It keeps the indentation, although you won't get
> > the compiler checking.
>
> After some reflection, I like this approach. It keeps the indentation
> and enables syntax highlighting, so it makes some of these blocks much
> easier to read.
Quite a few other places use this pattern:
#ifdef NOT_USED
Oid subtype = PG_GETARG_OID(3);
#endif
Nothing #define's NOT_USED, so it works the same, but helps document
that the argument isn't used.
Regards,
Dean
