Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> This patch is mainly doing four things:
> 1. take some typedefs and the HeapTupleData struct definition from
> access/htup.h, and put them in access/tupbasics.h. This new file is
> used as #include in all headers instead of htup.h.
> I'm unsure about the "tupbasics.h" file name. I'm open to better ideas.
I'd be inclined to keep the name "htup.h" for the more widely used file,
and invent a new name for what we're splitting out of it. This should
reduce the number of changes needed, not only in our code but third
party code. Not sure if the new file could sanely be called
"htup_private.h"; it seems a bit widely used for that. Maybe "heap.h"?
Also, is there any reason to consider just moving those defs into
heapam.h, instead of inventing a new header? I'm not sure if there's
any principled distinction between heap.h and heapam.h, or any
significant differences between their sets of consumers.
The other changes all sound sane.
regards, tom lane