On Wed, Apr 14, 2010 at 5:05 AM, Sebastian Sjöberg <[email protected]> wrote: > On Apr 14, 2010, at 7:39 AM, Nick Mathewson wrote: > >> On Wed, Mar 24, 2010 at 6:21 AM, Sebastian Sjöberg >> <[email protected]> wrote: >>> Hi, >>> >>> The EVUTIL_CLOSESOCKET() macro requires you to include unistd.h in your >>> source for linux and I don't know what on windows. To ease portablilty >>> wouldn't it be nice if we either turned the macro into a function or >>> included the appropriate header where we define the macro? >>> >>> I'd be happy to provide a patch for whatever behavior you would prefer. >> >> Sounds plausible. We might as well turn it into a function: an extra >> function call is going to be cheap in comparison with the system call. > > Here's the patch. Not tested on windows though.
Thanks; applied! -- Nick *********************************************************************** To unsubscribe, send an e-mail to [email protected] with unsubscribe libevent-users in the body.
