From: | "Merlin Moncure" <mmoncure(at)gmail(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Andrew Dunstan" <andrew(at)dunslane(dot)net>, "Alvaro Herrera" <alvherre(at)commandprompt(dot)com>, "Patches (PostgreSQL)" <pgsql-patches(at)postgresql(dot)org>, "Andrew Chernow" <ac(at)esilo(dot)com> |
Subject: | Re: libpq object hooks |
Date: | 2008-05-15 15:52:21 |
Message-ID: | b42b73150805150852m7b3c28fai2d7a4497c84118b2@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
On Wed, May 14, 2008 at 10:44 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> I'm wondering why the hooks need names at all. AFAICS all that
> libpq needs to know about a hook is a callback function address
> and a void * passthrough pointer.
I'm trying to make this work as you suggest.
It's pretty clear that all the callbacks should be modified to send a
void* along with the other arguments. This would eliminate the need
for hookname there (getting the state inside callback functions).
The problem is the functions PQhookData(conn, hookname) and
PQresultHookData(result, hookName). We need these to work in
functions that are not callbacks. If we eliminate hookname
completely, there is no way for libpq to know which private state we
are asking for. I'm a little bit stuck here. Is it possible to
preserve the hookname outside of the context of the callback functions
or is there something else I'm missing? (I'm sorry if I'm being
obtuse)
Eliminating the need for libpqtypes to need PQhookx functions, while
possible, would make libpqtypes a lot more difficult to use and
generally more awkward.
merlin
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-05-15 16:01:36 | Re: [rfc,patch] PL/Proxy in core |
Previous Message | Zdenek Kotala | 2008-05-15 15:48:58 | Re: bloated heapam.h |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2008-05-15 15:54:47 | Re: Patch to change psql default banner v6 |
Previous Message | Bruce Momjian | 2008-05-15 15:46:41 | Re: Patch to change psql default banner v6 |