Re: libpq WSACleanup is not needed

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Andrew Chernow <ac(at)esilo(dot)com>, James Mansion <james(at)mansionfamily(dot)plus(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: libpq WSACleanup is not needed
Date: 2009-01-20 15:09:51
Message-ID: 4975E93F.6070703@hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Bruce Momjian wrote:
> Andrew Chernow wrote:
>> Bruce Momjian wrote:
>>> We could have gone with a more elegant init/uninit solution but there is
>>> a history of slow upstream adoption of libpq API changes.
>>>
>>>
>> If that's the case, adding a connectdb option seems like a good
>> alternative. Orignally suggested here:
>>
>> http://archives.postgresql.org/pgsql-hackers/2009-01/msg01358.php
>
> Right, well the big question is how many people are going to use the
> connection option vs. doing it for everyone automatically.
>
> One possible approach might be to do it automatically, and allow a
> connection option to disable the WSACleanup() call.

I think that was the suggestion. Have an option that would disable
*both* the startup and the cleanup call, leaving the responsibility to
the app.

You can do this for SSL today by calling PQinitSSL().

> Actually, right now, if you have two libpq connections, and close one,
> does WSACleanup() get called, and does it affect the existing
> connection?

WSACleanup() gets called, but it has an internal reference count so it
does not have any effect on existing connections.

//Magnus

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2009-01-20 15:17:23 Re: FWD: Re: Updated backslash consistency patch
Previous Message Magnus Hagander 2009-01-20 14:51:16 Re: [BUGS] BUG #4186: set lc_messages does not work