On Mon, Feb 8, 2010 at 3:05 PM, Alvaro Herrera
<alvherre@commandprompt.com> wrote:
> 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.
I don't think it's inappropriate; it doesn't seem implausible that
someone might want to free an rbtree someday. I suppose we could
comment it out but I guess I don't see the point.
> How do we now that it works?
Visual inspection? It's not very complicated.
> (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)
Well, then you could have the freefunc do something ((MyStruct *)
a)->is_allocated = false.
...Robert