On Apr 14, 2010, at 7:39 AM, Nick Mathewson wrote: > On Wed, Mar 24, 2010 at 6:21 AM, Sebastian Sjöberg > <sebastian.sjob...@axis.com> 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, Sebastian Sjöberg
evutil_closesocket.patch
Description: evutil_closesocket.patch
> > -- > Nick > *********************************************************************** > To unsubscribe, send an e-mail to majord...@freehaven.net with > unsubscribe libevent-users in the body.