apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: cvs commit: apr/network_io/win32 sockets.c sockopt.c
Date Tue, 04 Nov 2003 08:19:40 GMT
On Tue, Nov 04, 2003 at 02:23:59AM +0100, Branko ?ibej wrote:
> gstein@apache.org wrote:
>...
> >  Also simplified the test/set macros to just take a socket.
>...
> >   #define apr_set_option(mask, option, on) \
> >       do {                                 \
> >           if (on)                          \
> >  -            *mask |= option;             \
> >  +            *(mask) |= (option);         \
> >           else                             \
> >  -            *mask &= ~option;            \
> >  +            *(mask) &= ~(option);        \
> >       } while (0)
> >   
> >   #endif  /* ! NETWORK_IO_H */
>
> Any particular reason for this macro not being the same as the Unix
> version, except making an evil attack on Win32 buildability? :-)

Gah... dealing with too many copies of too many files... Sorry 'bout that.

As you guessed, there was no particular reason. I just missed it and
couldn't get a test win32 compile done. Thanks for the fix!

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

Mime
View raw message