← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/net-boost-asio into lp:widelands

 

OK you use some struct ifaddrs from 
$FreeBSD: src/include/ifaddrs.h,v 1.3.32.1.4.1 2010/06/14 02:09:06 kensmith Exp $ 
As OSX is bases on Freebsd no surprise here.
man getifaddr tells me 

     The ifaddrs structure contains at least the following entries:

         struct ifaddrs   *ifa_next;         /* Pointer to next struct */
         char             *ifa_name;         /* Interface name */
         u_int             ifa_flags;        /* Interface flags */
         struct sockaddr  *ifa_addr;         /* Interface address */
         struct sockaddr  *ifa_netmask;      /* Interface netmask */
         struct sockaddr  *ifa_dstaddr;      /* P2P interface destination */
         void             *ifa_data;         /* Address specific data */

...
    Note that as a convenience, ifa_broadaddr is defined by a compiler 
    #define directive to be the same as ifa_dstaddr.

On linux this is:

         struct ifaddrs  *ifa_next;    /* Next item in list */
         char            *ifa_name;    /* Name of interface */
         unsigned int     ifa_flags;   /* Flags from SIOCGIFFLAGS */
         struct sockaddr *ifa_addr;    /* Address of interface */
         struct sockaddr *ifa_netmask; /* Netmask of interface */
         union {
             struct sockaddr *ifu_broadaddr;
                              /* Broadcast address of interface */
             struct sockaddr *ifu_dstaddr;
                              /* Point-to-point destination address */
         } ifa_ifu;
     #define              ifa_broadaddr ifa_ifu.ifu_broadaddr
     #define              ifa_dstaddr   ifa_ifu.ifu_dstaddr
         void            *ifa_data;    /* Address-specific data */

No ifa_ifu here, but as of the Note for BSD this should be ifa_broadaddr ...

Ok, this did the trick, will cehck in this chanage and some struct/class 
confusions my compiler complains about.

Notabilis: will you merge with trunk as Gun has commited some major change today?



-- 
https://code.launchpad.net/~widelands-dev/widelands/net-boost-asio/+merge/324364
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/net-boost-asio into lp:widelands.


References