Patch for a bug in server/select.c
Bang Jun-Young
bjy at mogua.org
Sat May 19 00:24:12 CDT 2001
On Fri, May 18, 2001 at 06:42:59PM -0000, Morten Welinder wrote:
>
> > +#ifdef __NetBSD__
> > + if (pollfd[i].revents & !(pollfd[i].revents & POLLNVAL))
> > +#else
> > if (pollfd[i].revents)
> > +#endif
>
> I would hate to see this kind of __NetBSD__ pollution of Wine.
> (I believe something like it slipped through recently -- not
> good.)
>
> It used to be that documentation/porting contained lots of reasons
> why mentioning any OS name in places like this is a bad idea.
>
> I suggest a suitable configure test or a porting wrapper for poll.
I believe stripping __NetBSD__ #if/endif pair doesn't any harm in any
cases, but just a little bit of performance degradation. Which one
should we choose between readability, portability, or performance?
Jun-Young
--
Bang Jun-Young <bjy at mogua.org>
More information about the wine-devel
mailing list