[vlc-devel] Re: [patch] IGMPv3 support for Linux

[Gildas Bazin]

On Sunday 01 February 2004 02:15, Alexis de Lattre wrote:
> 
> So I decided to put :
> 
> #if defined( SYS_LINUX )
>     struct ip_mreq_source {
>         struct in_addr  imr_multiaddr;
>         struct in_addr  imr_interface;
>         struct in_addr  imr_sourceaddr;
>      };
> 
>     #define IP_ADD_SOURCE_MEMBERSHIP  39
> #endif
> 

Personnaly I'd rather have:

#ifndef IP_ADD_SOURCE_MEMBERSHIP
     #define IP_ADD_SOURCE_MEMBERSHIP  39
     struct ip_mreq_source {
         struct in_addr  imr_multiaddr;
         struct in_addr  imr_interface;
         struct in_addr  imr_sourceaddr;
      };
#endif

I suppose it shouldn't create a problem because  struct ip_mreq_source 
shouldn't be defined already if IP_ADD_SOURCE_MEMBERSHIP isn't.
Rocky's solution would be fine as well however a bit more complicated.

--
Gildas

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html
If you are in trouble, please contact <postmaster at videolan.org>