[FFmpeg-devel] [PATCH] Enable proper IPv6 functions on windows

Reimar Döffinger Reimar.Doeffinger
Sat Jan 16 15:16:00 CET 2010


On Sat, Jan 16, 2010 at 03:58:50PM +0200, Martin Storsj? wrote:
> On Sat, 16 Jan 2010, Reimar D?ffinger wrote:
> 
> > On Sat, Jan 16, 2010 at 11:42:56AM +0200, Martin Storsj? wrote:
> > > Yes, your approach is much simpler in that sense, and shouldn't have any 
> > > thread safety issues either. GetModuleHandle instead of LoadLibrary should 
> > > be ok, too, since we actually link to that dll. The only drawback is that 
> > > it pollutes the generic fallback wrappers with platform specific code.
> > 
> > I don't dispute it's ugly, but the configure changes don't exactly look
> > non-ugly to me either while giving much worse usability.
> 
> Ok, so here's the two options:
> 
> win-ipv6-load.patch adds a winsock specific GetModuleHandle + 
> GetProcAddress at the beginning of each of the wrapper fallbacks, making 
> it backwards compatible while giving the new functionality on newer 
> versions. If this path is chosen, commit 21227 can (should?) be reverted.
> 
> Reimar, does this one look sensible to you?

I see no obvious problems with it.

> 0001... first changes the current addition of the _WIN32_WINNT 
> define to only be added if target-os = mingw32, for other ones (mingw32ce, 
> mingw32-prexp or mingw32-foobar or whatever), no define is added. The 
> follow-up, 0002 checks for getaddrinfo in ws2tcpip.h, which succeeds 
> or fails depending on the target-os selection.

If others insist on this, it _might_ have better usability to just add
the define and enable it when --enable-ipv6 is explicitly specified or something
like that...



More information about the ffmpeg-devel mailing list