Robert Haas <robertmhaas@gmail.com> writes:
> Here is a patch that *just* introduces planstate_tree_walker and which
> is hopefully correct. I stole the logic from ExplainPreScanNode,
> which I also refactored to use the new walker instead of duplicating
> the logic.
It seems a little odd to have removed functions from explain.c altogether
but left their header comments behind. Otherwise this seems sound.
regards, tom lane