Re: pgsql: Add support for coordinating record typmods among parallel worke

Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> writes:
> My compiler, C++ and more recent C standards are OK with identical
> redefinition of a typedef like that, but not the older standards or
> those particular compilers. D'oh. (I should figure out how to make
> my automatic patch tester this fussy). I think we should probably
> just do this:

Our usual locution for this sort of thing is to use
"struct SharedRecordTypmodRegistry" in headers instead of
the typedef name, if we don't want to pull in the header
where the typedef is defined. It might be all right to do
what you suggest, but we've been burnt in the past by circular
dependencies and/or pulling some header into essentially
the entire build (which tends to lead to a mess down the road).
I'd generally lean in the direction of not adding #includes to
header files except where absolutely necessary.

regards, tom lane