Re: DatumGetInetP buggy

Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> writes:
> Hmm, it seems to be intentional, but I agree it's very much contrary to
> the usual convention that DatumGetXXXP() returns a detoasted and
> depacked datum. I think we should change it. I propose the attached
> patch. It changes DatumGetInetP() to do PG_DETOAST_DATUM(), and adds new
> DatumGetInetPP() macro to return the packed version. I also moved the
> access macros like ip_family() from network.c to inet.h, so that they're
> available for whoever wants to look at the fields without having to depack.

No objection to making the DatumGet macro names conform to common
convention, but I'm not thrilled with moving those special-purpose
accessor macros into wider circulation. It's not necessary and the
macros don't work unless used in a particular way per the comment,
so I don't think they can be considered general purpose.

regards, tom lane