Re: split TOAST support out of postgres.h

Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> writes:
> ... Then we could either

> 1) Include varatt.h in postgres.h, similar to elog.h and palloc.h. That
> way we clean up the files a bit but don't change any external interfaces.

> 2) Just let everyone who needs it include the new file.

> 3) Compromise: You can avoid most "damage" by having fmgr.h include
> varatt.h. That satisfies most data types and extension code. That way,
> there are only a few places that need an explicit include of varatt.h.

> I went with the last option in my patch.

I dunno, #3 seems kind of unprincipled. Also, since fmgr.h is included
so widely, I doubt it is buying very much in terms of reducing header
footprint. How bad is it to do #2?

regards, tom lane