It seems a bit strange to have all the rb_free_recursive support and not
use it anywhere ... and a freefunc callback even, whose only caller
seems to set as null currently.  Hmm, even in the knngist patch the
rb_freefunc stuff is unused.  How do we now that it works?

(What, for example, if we were to allocate multiple nodes in a single
palloc chunk?  I'm not familiar with this stuff but that seems
plausible)

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to