[PATCH xserver 17/23] os: Add ospoll interface
jcristau at debian.org
Mon May 30 06:38:49 UTC 2016
On Sun, May 29, 2016 at 11:35:16 -0700, Keith Packard wrote:
> Emil Velikov <emil.l.velikov at gmail.com> writes:
> > There's two things which I should have mentioned:
> > - How does one have poll.h when poll() is missing - Cygwin may have
> > the header, although I'm not sure about other Windows based
> > implementations (the mingw and msvc ones).
> I'd expect whatever changes were needed to support those operating
> systems to come from people using them. In particular, Windows has
> winsock2.h, which may provide the necessary values, but as I'm not doing
> that port, I can't really say what changes should be provided. I'd love
> to get feedback from people who can provide it.
> > - Considering that we have our own header with wrapper functions and
> > an opaque struct, it would make sense to have X server specific
> > macros. Even (ideally) if they are ABI compatible with the POSIX ones.
> Yeah, maybe just using X_NOTIFY_READ, X_NOTIFY_WRITE instead of
> How about:
> #define X_NOTIFY_READ POLLIN
> #define X_NOTIFY_WRITE POLLOUT
> #define X_NOTIFY_OTHER (!(POLLIN|POLLOUT))
I guess you mean ~(POLLIN|POLLOUT).
More information about the xorg-devel