Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-08 Thread Josh Triplett
On Wed, Nov 06, 2013 at 07:09:01PM -0800, Keith Packard wrote: Peter Harris phar...@opentext.com writes: Dude. It's not getting NAKed, it's getting Woah, this is out of left field, this is the first I've ever heard of it, can we please have a couple of days to think about it before we ACK

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-08 Thread Josh Triplett
On Wed, Nov 06, 2013 at 09:25:40PM -0500, Peter Harris wrote: On 2013-11-06 19:55, Josh Triplett wrote: On Wed, Nov 06, 2013 at 02:34:03PM -0500, Peter Harris wrote: On 2013-11-05 19:41, Keith Packard wrote: This allows apps to peel off certain XGE events into separate queues for custom

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-08 Thread Josh Triplett
On Wed, Nov 06, 2013 at 06:39:19PM -0800, Keith Packard wrote: Josh Triplett j...@joshtriplett.org writes: The last time that Jamey and I spoke about this, we agreed that this type of mechanism would be required to make GL work, and that there's really no other way to support it.

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-07 Thread Uli Schlachter
Hi, On 07.11.2013 04:09, Keith Packard wrote: Peter Harris phar...@opentext.com writes: Dude. It's not getting NAKed, it's getting Woah, this is out of left field, this is the first I've ever heard of it, can we please have a couple of days to think about it before we ACK it?ed. Yeah,

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-07 Thread Keith Packard
Uli Schlachter psyc...@znc.in writes: And this comes up on the xcb list two days before this deadline and only does so accidentally...? Well, many of us appear to forget that xcb has a magic special list, separate from xorg-devel... If this patch does get in, I am OK with cherry-picking

Re: [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Peter Harris
On 2013-11-05 19:41, Keith Packard wrote: This allows apps to peel off certain XGE events into separate queues for custom handling. Designed to support the Present extension Signed-off-by: Keith Packard kei...@keithp.com Re: xcb_register_for_special_event, xcb_unregister_for_special_event,

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Uli Schlachter
Hi, On 06.11.2013 20:34, Peter Harris wrote: On 2013-11-05 19:41, Keith Packard wrote: [...] diff --git a/src/xcb.h b/src/xcb.h index f7dc6af..f99e677 100644 --- a/src/xcb.h +++ b/src/xcb.h @@ -138,23 +138,6 @@ typedef struct { } xcb_generic_event_t; /** - * @brief GE event - * -

Re: [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Keith Packard
Peter Harris phar...@opentext.com writes: There was some push-back the last time something similar to this came up, on the theory that it should be in a separate library from xcb[1]. That said, we may have reached the point where it's just too useful to not have in libxcb. We cannot

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Keith Packard
Uli Schlachter psyc...@znc.in writes: With google I found the patches in question and looked at them quickly. My first impression is that the xcb_*_special_event() functions might need way better documentation. That's always true :-) Also, this adds public API that allows to hide XGE

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Eric Anholt
Uli Schlachter psyc...@znc.in writes: Hi, On 06.11.2013 20:34, Peter Harris wrote: On 2013-11-05 19:41, Keith Packard wrote: [...] diff --git a/src/xcb.h b/src/xcb.h index f7dc6af..f99e677 100644 --- a/src/xcb.h +++ b/src/xcb.h @@ -138,23 +138,6 @@ typedef struct { }

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Josh Triplett
On Wed, Nov 06, 2013 at 02:34:03PM -0500, Peter Harris wrote: On 2013-11-05 19:41, Keith Packard wrote: This allows apps to peel off certain XGE events into separate queues for custom handling. Designed to support the Present extension Signed-off-by: Keith Packard kei...@keithp.com

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Eric Anholt
Peter Harris phar...@opentext.com writes: On 2013-11-05 19:41, Keith Packard wrote: This allows apps to peel off certain XGE events into separate queues for custom handling. Designed to support the Present extension Signed-off-by: Keith Packard kei...@keithp.com Re:

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Peter Harris
On 2013-11-06 19:55, Eric Anholt wrote: We tried to do this with event queue filtering the same way we do with xlib, and it got NAKed. Then we came up with new events and a new mechanism so that we could do this, and it's getting NAKed again. Dude. It's not getting NAKed, it's getting Woah,

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Peter Harris
On 2013-11-06 19:55, Josh Triplett wrote: On Wed, Nov 06, 2013 at 02:34:03PM -0500, Peter Harris wrote: On 2013-11-05 19:41, Keith Packard wrote: This allows apps to peel off certain XGE events into separate queues for custom handling. Designed to support the Present extension Signed-off-by:

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Keith Packard
Josh Triplett j...@joshtriplett.org writes: The last time that Jamey and I spoke about this, we agreed that this type of mechanism would be required to make GL work, and that there's really no other way to support it. However, I'd like to see it not XGE-specific, so that it could support the

Re: [Xcb] [PATCH 5/7] Add event queue splitting

2013-11-06 Thread Keith Packard
Peter Harris phar...@opentext.com writes: Dude. It's not getting NAKed, it's getting Woah, this is out of left field, this is the first I've ever heard of it, can we please have a couple of days to think about it before we ACK it?ed. Yeah, I've been running this code for months now and have

[PATCH 5/7] Add event queue splitting

2013-11-05 Thread Keith Packard
This allows apps to peel off certain XGE events into separate queues for custom handling. Designed to support the Present extension Signed-off-by: Keith Packard kei...@keithp.com --- src/xcb.h| 45 ++-- src/xcb_in.c | 167