"Brendan Jurd" <direvus@gmail.com> writes:
> So, it looks like:
> 1. describe malloc's the cells to zero, but print just does a local
> calloc without any initialisation.
There isn't any functional difference there. I am not sure, but I think
the reason print.c has its own malloc wrappers instead of depending on
common.c's is that we use print.c in some bin/scripts/ programs that
do not want common.c too.
> 2. describe only does an mbvalidate for WIN32, but print does it in all cases.
I don't know why describe only does that for WIN32; it looks
inconsistent to me too. Possibly some trolling in the CVS history would
give a clue about this.
If you're not actively working on this patch right now, I am going to go
ahead and commit the other open patches for describe.c. If you do have
a patch in progress, I'm willing to hold off to avoid any merge
conflicts. Let me know.
regards, tom lane