From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, Mark Cave-Ayland <mark(dot)cave-ayland(at)siriusit(dot)co(dot)uk>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [CFReview] Red-Black Tree |
Date: | 2010-02-09 04:40:33 |
Message-ID: | 603c8f071002082040u20db6446pfdf45ffc571dbd7f@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Feb 8, 2010 at 10:43 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> On Mon, Feb 8, 2010 at 3:05 PM, Alvaro Herrera
>> <alvherre(at)commandprompt(dot)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.
>
> I think the suggestion was to *remove* it not comment it out. I'm
> skeptical of carrying dead code. If the functionality is not used
> in the proposed gist patches then it's very fair to question whether
> it ever will be used.
I don't think the question is unfair; I just don't happen to agree
with the conclusion. But I don't care enough to argue about it
either...
...Robert
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2010-02-09 04:43:08 | Re: [CFReview] Red-Black Tree |
Previous Message | Bruce Momjian | 2010-02-09 04:32:31 | Re: Hot standby documentation |