Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> Here is a patch that moves two blocks from builtins.h into separate
> header files. That avoids having to include nodes/pg_list.h and
> utils/sortsupport.h.
Seems like a reasonable solution.
> The remaining inclusion of nodes/nodes.h is for the oidparse() function.
> I think that could be moved out of oid.c and put somewhere near parser/
> or objectaddress.c. But the practical savings from avoiding nodes.h is
> probably near zero, so I haven't done anything about that here.
Agreed, not much value in that.
regards, tom lane