On Fri, May 16, 2008 at 3:49 PM, Merlin Moncure <[EMAIL PROTECTED]> wrote: > On Fri, May 16, 2008 at 2:34 PM, Andrew Chernow <[EMAIL PROTECTED]> wrote: >> Tom Lane wrote: >>> >>> typedef void (*PGeventProc) (PGeventId eventId, const void *eventInfo, >>> void *passthrough); >>> >>> int PQregisterEventProc(PGconn *conn, PGeventProc proc, void >>> *passthrough); > >> The above prototypes will work and we will add our 'event instance pointer' >> to the event info structures. Should have a patch shortly. > > > Right. I actually overlooked the 'passthrough' in > PQregisterEventProc. It turns out that we are still not quite on the > same page and this needs to be clarified before we move on. The > passthrough cannot exist...the correct prototypes (reasoning will > follow) are: > > typedef void (*PGeventProc) (PGeventId eventId, const void *eventInfo);
small typo: eventInfo obviously can't be const > int PQregisterEventProc(PGconn *conn, PGeventProc proc); > PQhookData(const PGconn* conn, PGeventProc proc); PQhookData is the old name...we are going with 'events' now....the proper names will come with the patch. merlin -- Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-patches