Re: libpq WSACleanup is not needed - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: libpq WSACleanup is not needed
Date
Msg-id 200901201703.n0KH38320616@momjian.us
Whole thread Raw
In response to Re: libpq WSACleanup is not needed  (Andrew Chernow <ac@esilo.com>)
List pgsql-hackers
Andrew Chernow wrote:
> Bruce Momjian wrote:
> > Andrew Chernow wrote:
> >> Bruce Momjian wrote:
> >>> Andrew Chernow wrote:
> >>>> Bruce Momjian wrote:
> >>>>> Ah, OK, so it does its own cleanup on last close, great. I agree a
> >>>>> connection option for this would be good.
> >>>>>
> >>>> What would the option be?  "wsainit = [enable | disable]"?  Maybe it 
> >>>> should allow setting the version to load: "wsa_version = 2.0".  Maybe 
> >>>> the two should be combined: "wsa_version = [default | disable | 2.0]".
> >>> I assumed it would be like SSL, which is a libpq function call, not a
> >>> connection option, e.g. PQinitSSL(), and I think true/false is probably
> >>> enough.  PQinitSSL info:
> >>>
> >>>    If you are using <acronym>SSL</> inside your application (in addition
> >>>    to inside <application>libpq</application>), you can use
> >>>    <function>PQinitSSL(int)</> to tell <application>libpq</application>
> >>>    that the <acronym>SSL</> library has already been initialized by your
> >>>    application.
> >>>
> >> That smells dirty to me.  How many PQinitXXX() functions are needed 
> >> before we drop the XXX and run with PQinit(...)?
> > 
> > Odds are you would still need per-library control over initialization so
> > I am not sure that helps, i.e. the library initialized WSA already but
> > needs SSL.
> > 
> 
> That's fine.  I solved that issue here:
> 
> http://archives.postgresql.org/pgsql-hackers/2009-01/msg01349.php
> 
> One of arguments is an "options" bit mask.  PG_OPT_LOADSSL, 
> PG_OPT_LOADWSA, etc...  I also suggested a "int inittype, void 
> *initdata" arguments that can be used now or for future expansion; that 
> way PQinit is not limited to a single int argument.  This could be used 
> right away with the PG_OPT_LOADWSA idea, to pass the wsa version you want.

That seems overly complex to support just two init functions (we only
had one for SSL for years).

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +


pgsql-hackers by date:

Previous
From: Andrew Chernow
Date:
Subject: Re: libpq WSACleanup is not needed
Next
From: Heikki Linnakangas
Date:
Subject: Re: visibility maps and heap_prune