Alvaro Herrera <alvherre@commandprompt.com> writes:
> I propose the following patch which moves the struct definitions to a
> separate new header relscan_internal.h.
This seems a little bizarre, seeing that there is almost nothing in
relscan.h except those structs.
Perhaps a better idea would be to put the opaque-pointer typedefs into
heapam.h and genam.h respectively, and then see where you could remove
inclusions of relscan.h.
Also, it seemed like some of those .c files had no business poking into
the scan structs anyway; particularly contrib. Did you check whether
the inclusions could be avoided?
regards, tom lane