From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
Cc: | Boszormenyi Zoltan <zb(at)cybertec(dot)at>, pgsql-hackers(at)postgresql(dot)org, Hans-Juergen Schoenig <hs(at)cybertec(dot)at>, Leonardo Bispo de Oliveira <leonardo(dot)oliveira(at)voipfuture(dot)com> |
Subject: | Re: DatumGetInetP buggy |
Date: | 2011-11-08 15:06:26 |
Message-ID: | 20757.1320764786@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2011-11-08 15:08:53 | Re: heap vacuum & cleanup locks |
Previous Message | Marko Kreen | 2011-11-08 15:05:46 | Re: Disable OpenSSL compression |