On 2020-Mar-05, Tom Lane wrote:
> Alvaro Herrera <alvherre@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?
Good idea, thanks! Pushed with that change.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services