From: Michael Paquier <michael@paquier.xyz>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Clean up some code, comments and docs referring to Windows 2000
Date: Thu, 20 Feb 2020 12:02:40 +0900
Message-ID: <20200220030240.GG2288@paquier.xyz> (raw)
In-Reply-To: <9781.1582146114@sss.pgh.pa.us>
References: <E1j4GrA-0006wL-WE@gemulon.postgresql.org>
<9781.1582146114@sss.pgh.pa.us>
On Wed, Feb 19, 2020 at 04:01:54PM -0500, Tom Lane wrote:
> Surely this patch broke the error case in haveNativeWindowsIPv6routines()?
> That is, in the admittedly unlikely case that LoadLibraryA("ws2_32")
> succeeds but GetProcAddress(hLibrary, "getaddrinfo") doesn't, what you
> now have will do FreeLibrary(hLibrary) and then proceed to use the
> now-dangling hLibrary pointer anyway.
You are right, thanks! This block has no meaning anymore, so
removed.
For the note, it looks that we could get completely rid of
src/port/getaddrinfo.c once we drop support for XP as IPv6 support is
enabled by default on Vista and Windows 7. For XP, one needs to
install an extra module manually. Do you think that there are other
platforms using our port implementation for getaddrinfo() except
Windows? That's not material for today, obviously.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (832B, ../20200220030240.GG2288@paquier.xyz/2-signature.asc)
download
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-committers@postgresql.org
Cc: michael@paquier.xyz, tgl@sss.pgh.pa.us, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Clean up some code, comments and docs referring to Windows 2000
In-Reply-To: <20200220030240.GG2288@paquier.xyz>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox