[vlc-devel] [PATCH] Contrib: fix UPNP building on Win32

[Steve Lhomme]

On Mon, Mar 2, 2015 at 4:41 PM, Steve Lhomme <robux4 at videolabs.io> wrote:
> On Mon, Mar 2, 2015 at 4:32 PM, Jean-Baptiste Kempf <jb at videolan.org> wrote:
>> On 02 Mar, Steve Lhomme wrote :
>>> Fixes #14057
>>> ---
>>>  modules/services_discovery/upnp.hpp | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>
>>> diff --git a/modules/services_discovery/upnp.hpp b/modules/services_discovery/upnp.hpp
>>> index 23fe4db..09e1bc3 100644
>>> --- a/modules/services_discovery/upnp.hpp
>>> +++ b/modules/services_discovery/upnp.hpp
>>> @@ -28,6 +28,10 @@
>>>  #include <vector>
>>>  #include <string>
>>>
>>> +#ifdef _WIN32
>>> +# undef if_nametoindex
>>> +#endif
>>
>> Your mingw header is declaring if_nametoindex with the wrong WIN32_WINNT
>> guards.
>
>
> No it's guarded with _WIN32_WINT >= 0x600, which corresponds to the
> availability in Windows.
> https://msdn.microsoft.com/en-us/library/windows/desktop/bb408409%28v=vs.85%29.aspx

You're right I was looking at the one detected by QtCreator, which is
correct. The one in my mingw32 folder is wrong.

It comes from this file that is recommended in the Win32 compile instructions.
http://sourceforge.net/projects/mingw-w64/files/Toolchains%20targetting%20Win32/Personal%20Builds/mingw-builds/4.9.2/threads-win32/sjlj/i686-4.9.2-release-win32-sjlj-rt_v3-rev0.7z/download

Do we still need this hack ? There's a newer version of this package,
but the netioapi.h file is still bogus.
http://sourceforge.net/projects/mingw-w64/files/Toolchains%20targetting%20Win32/Personal%20Builds/mingw-builds/4.9.2/threads-win32/sjlj/i686-4.9.2-release-win32-sjlj-rt_v3-rev0.7z/download