Alvaro Herrera <alvhe...@2ndquadrant.com> writes:
> Thanks -- ISTM it makes more sense to put the FmgrInfo before the
> typioparam too:

> typedef struct RangeIOData
> {
>     TypeCacheEntry *typcache;    /* range type's typcache entry */
>     FmgrInfo    proc;            /* element type's I/O function */
>     Oid         typioparam;      /* element type's I/O parameter */
> } RangeIOData;

Yeah, WFM.  Maybe even rename the FmgrInfo to "typioproc"
or the like?

                        regards, tom lane


Reply via email to