"Brendan Jurd" <[EMAIL PROTECTED]> 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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers