Re: [PATCH:xmag] Use lrint() from math library if available

2011-12-20 Thread James Cloos
> "AC" == Alan Coopersmith writes: AC> Moves -lm from being hardcoded in Makefile.am to being added via AC> AC_SEARCH_LIBS in configure.ac setting it in $(LIBS) AC> Using lrint() [returns long int] instead of rint() [returns double] AC> clears a bunch of gcc warnings of the form: AC> "cast f

[PULL] XI 2.2 and multitouch patches

2011-12-20 Thread Peter Hutterer
Keith, Please pull the XI 2.2 work into master. This bumps the input ABI to 15, but I also hope that we don't need another bump this cycle. Carlos found the one known outstanding active grab bug that has now been fixed. This patchset should not affect normal pointer/keyboard behaviour, any such b

Re: [PATCH multitouch] Xi: only activate the device grab if we don't already have one

2011-12-20 Thread Chase Douglas
On Dec 20, 2011, at 8:47 PM, Peter Hutterer wrote: > If the device is already grabbed, don't activate the passive grab, it screws > with our event masks. Just deliver to the grabbing client instead. > > Reported-by: Carlos Garnacho > Signed-off-by: Peter Hutterer Makes sense to me. Reviewed-

[PATCH:xmag] Use lrint() from math library if available

2011-12-20 Thread Alan Coopersmith
Moves -lm from being hardcoded in Makefile.am to being added via AC_SEARCH_LIBS in configure.ac setting it in $(LIBS) Using lrint() [returns long int] instead of rint() [returns double] clears a bunch of gcc warnings of the form: "cast from function call of type ‘double’ to non-matching type ‘shor

[PATCH multitouch] Xi: only activate the device grab if we don't already have one

2011-12-20 Thread Peter Hutterer
If the device is already grabbed, don't activate the passive grab, it screws with our event masks. Just deliver to the grabbing client instead. Reported-by: Carlos Garnacho Signed-off-by: Peter Hutterer --- Xi/exevents.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH GL 1/1] XQuartz: GL: Buildfix for recent GLX changes

2011-12-20 Thread Jeremy Huddleston
dispatch.h was leftover from an earlier implementation and is no longer needed, so remove it since including it causes a build failure due to conflicts between GL/gl.h and OpenGL/gl.h Signed-off-by: Jeremy Huddleston --- hw/xquartz/GL/indirect.c |3 --- 1 files changed, 0 insertions(+), 3 de

Re: xserver: Branch 'master' - 5 commits

2011-12-20 Thread Jeremy Huddleston
On Dec 20, 2011, at 3:39 PM, Ian Romanick wrote: > On 12/20/2011 03:22 PM, Jeremy Huddleston wrote: >> It looks like this is causing a build regression on the tinderbox: >> http://tinderbox.x.org/builds/2011-12-20-0004/logs/xserver/#build >> >> It looks odd, so I'll take a look at it tonight, bu

Re: [PATCH util-macros 1/1] Provide a configure option to turn off the selective -Werror=... CFLAGS

2011-12-20 Thread Alan Coopersmith
On 12/19/11 18:50, Jeremy Huddleston wrote: Provide end users with a workaround if they find code doesn't build with the -Werror=... flags provided by BASE_CFLAGS. This can be useful if newer util-macros are used with older source code that had problems, or in the rare case when warnings are exp

Re: [PATCH v2 31/42] Process and deliver touch events

2011-12-20 Thread Peter Hutterer
On Tue, Dec 20, 2011 at 04:32:52PM -0800, Chase Douglas wrote: > From: Peter Hutterer > > Does not include pointer emulation handling. > Does include partial ownership handling but not the actual processing of > ownership events. > > Note: this commit is a retroactive commit extracted from a ser

Re: [PATCH multitouch] dix: remove requirement for client_id be the first element

2011-12-20 Thread Chase Douglas
On 12/20/2011 05:16 PM, Peter Hutterer wrote: > Leftover code from an earlier version of GetTouchEvents. > > Signed-off-by: Peter Hutterer > --- > As mentioned in another email. This is just cleanup without a functional > change. The client_id as first element was a requirement in earlier version

Re: [PATCH 20/42] dix: handle DIX-submitted touch events

2011-12-20 Thread Chase Douglas
On 12/20/2011 05:14 PM, Peter Hutterer wrote: > On Mon, Dec 19, 2011 at 06:33:06PM -0800, Chase Douglas wrote: >> On Dec 14, 2011, at 7:01 PM, Peter Hutterer wrote: >>> The DIX needs to submit touch events for e.g. TouchEnd after an >>> acceptance/rejection. These have the TOUCH_CLIENT_ID flag set

[PATCH multitouch] dix: remove requirement for client_id be the first element

2011-12-20 Thread Peter Hutterer
Leftover code from an earlier version of GetTouchEvents. Signed-off-by: Peter Hutterer --- As mentioned in another email. This is just cleanup without a functional change. The client_id as first element was a requirement in earlier versions and got virtually removed except for the client_id part.

Re: [PATCH 20/42] dix: handle DIX-submitted touch events

2011-12-20 Thread Peter Hutterer
On Mon, Dec 19, 2011 at 06:33:06PM -0800, Chase Douglas wrote: > On Dec 14, 2011, at 7:01 PM, Peter Hutterer wrote: > > The DIX needs to submit touch events for e.g. TouchEnd after an > > acceptance/rejection. These have the TOUCH_CLIENT_ID flag set. > > > > Signed-off-by: Peter Hutterer > > ---

Re: [PATCH v2 37/42] Xi: add the hooks for passive touch grabs

2011-12-20 Thread Chase Douglas
On 12/20/2011 04:24 PM, Peter Hutterer wrote: > Co-authored-by: Daniel Stone > Signed-off-by: Peter Hutterer > --- > Changes to v1: > - check for mode XIGrabModeTouch on touch grabs Looks good! Reviewed-by: Chase Douglas ___ xorg-devel@lists.x.org: X

[PATCH v2 35/42] Xi: hook up touch ownership handling

2011-12-20 Thread Chase Douglas
From: Peter Hutterer Ownership changes don't get processed directly when they happen, instead the DIX submits an ownership event which goes through ProcessTouchEvents and ProcessTouchOwnershipEvents. Then on the required events are generated and sent to clients. Co-authored-by: Daniel Stone Sig

[PATCH v2 34/42] Xi: hook up pointer emulation for emulating touches

2011-12-20 Thread Chase Douglas
From: Peter Hutterer Includes a hack for implicit grab activation, because integrating this requires a larger rewrite and I'm not sleeping enough as it is. Right now, we deliver the event and check before/after if there is an implicit grab on. If one activated, then store the event in the grab an

[PATCH v2 31/42] Process and deliver touch events

2011-12-20 Thread Chase Douglas
From: Peter Hutterer Does not include pointer emulation handling. Does include partial ownership handling but not the actual processing of ownership events. Note: this commit is a retroactive commit extracted from a series of ~50 commits and may thus appear a bit more complicated than what you'd

[PATCH v2 37/42] Xi: add the hooks for passive touch grabs

2011-12-20 Thread Peter Hutterer
Co-authored-by: Daniel Stone Signed-off-by: Peter Hutterer --- Changes to v1: - check for mode XIGrabModeTouch on touch grabs Xi/exevents.c | 28 Xi/xipassivegrab.c | 17 +++-- include/exevents.h |8 3 files changed, 51 insertions(+

Re: [PATCH 24/42] dix: add helper functions to build up/verify the sprite trace

2011-12-20 Thread Peter Hutterer
On Tue, Dec 20, 2011 at 03:46:02PM -0800, Chase Douglas wrote: > On 12/20/2011 03:36 PM, Peter Hutterer wrote: > > On Wed, Dec 21, 2011 at 09:32:29AM +1000, Peter Hutterer wrote: > >> On Mon, Dec 19, 2011 at 07:01:25PM -0800, Chase Douglas wrote: > >>> On Dec 14, 2011, at 7:02 PM, Peter Hutterer

Re: [PATCH 24/42] dix: add helper functions to build up/verify the sprite trace

2011-12-20 Thread Chase Douglas
On 12/20/2011 03:36 PM, Peter Hutterer wrote: > On Wed, Dec 21, 2011 at 09:32:29AM +1000, Peter Hutterer wrote: >> On Mon, Dec 19, 2011 at 07:01:25PM -0800, Chase Douglas wrote: >>> On Dec 14, 2011, at 7:02 PM, Peter Hutterer >>> wrote: From: Daniel Stone Touch events' sprite trac

Re: [PATCH 24/42] dix: add helper functions to build up/verify the sprite trace

2011-12-20 Thread Peter Hutterer
On Wed, Dec 21, 2011 at 09:32:29AM +1000, Peter Hutterer wrote: > On Mon, Dec 19, 2011 at 07:01:25PM -0800, Chase Douglas wrote: > > On Dec 14, 2011, at 7:02 PM, Peter Hutterer > > wrote: > > > From: Daniel Stone > > > > > > Touch events' sprite trace stays the same for the duration of the touc

Re: [PATCH 24/42] dix: add helper functions to build up/verify the sprite trace

2011-12-20 Thread Peter Hutterer
On Mon, Dec 19, 2011 at 07:01:25PM -0800, Chase Douglas wrote: > On Dec 14, 2011, at 7:02 PM, Peter Hutterer wrote: > > From: Daniel Stone > > > > Touch events' sprite trace stays the same for the duration of the touch > > sequence. > > > > Signed-off-by: Peter Hutterer > > --- > > dix/touch.c

Re: xserver: Branch 'master' - 5 commits

2011-12-20 Thread Jeremy Huddleston
It looks like this is causing a build regression on the tinderbox: http://tinderbox.x.org/builds/2011-12-20-0004/logs/xserver/#build It looks odd, so I'll take a look at it tonight, but I wanted to bring it up. --Jeremy On Dec 19, 2011, at 10:52 PM, Keith Packard wrote: > glx/Makefile.am

Re: [PATCH 00/42] Multitouch support

2011-12-20 Thread Peter Hutterer
On Tue, Dec 20, 2011 at 11:53:41AM -0800, Chase Douglas wrote: > On 12/14/2011 07:01 PM, Peter Hutterer wrote: > > > > This is the set of patches to add multitouch support to the X server. > > The lot can be found in the 'multitouch' branches of the following > > repositories > > > > git://pe

Re: [PATCH 27/42] dix: add helper functions for adding/removing touch listeners

2011-12-20 Thread Peter Hutterer
On Tue, Dec 20, 2011 at 09:04:38AM -0800, Chase Douglas wrote: > On 12/19/2011 07:27 PM, Peter Hutterer wrote: > > On Mon, Dec 19, 2011 at 07:20:38PM -0800, Chase Douglas wrote: > >> On Dec 14, 2011, at 7:02 PM, Peter Hutterer > >> wrote: > >> > >> [snip] > >> > >>> +static void > >>> +TouchAddAc

Re: [PATCH 01/21] include: add GetXI2MaskByte and GetXI2EventFilterMask to headers

2011-12-20 Thread Peter Hutterer
On Tue, Dec 20, 2011 at 11:46:27AM -0500, Adam Jackson wrote: > On 12/8/11 10:36 PM, Peter Hutterer wrote: > > >@@ -414,7 +414,7 @@ GetEventFilterMask(DeviceIntPtr dev, int evtype) > > return filters[dev ? dev->id : 0][evtype]; > > } > > > >-static inline Mask > >+inline Mask > > GetXI2Even

[PULL] regression, bug fix

2011-12-20 Thread Peter Hutterer
7af23259d88f4c28ed21140f82cc03b3724c06bb introduced a regression, fixed in the first of the two patches. The second one is a rather long-standing bug that probably affects all XI2-capable servers and should land on 1.11. commit fcda98c48610fd507ca0b89c6006a5497d9dc1c9 The following changes since

Re: [PATCH inputproto multitouch] Reinstate libXi's version defines

2011-12-20 Thread Chase Douglas
On 12/19/2011 02:23 PM, Peter Hutterer wrote: > Realistically, we can't remove these from the protocol without breaking > older libraries. > > Introduced in a02566ca7fd37d279b957037e1251a3b3419866d > > Signed-off-by: Peter Hutterer > --- > XI.h | 11 +++ > 1 files changed, 11 insertio

Re: [PATCH inputproto multitouch] Drop wrong comment for sourceid in TouchOwnershipEvents

2011-12-20 Thread Chase Douglas
On 12/19/2011 02:22 PM, Peter Hutterer wrote: > Copy/paste error from DeviceChangedEvent > > Signed-off-by: Peter Hutterer > --- > XI2proto.h |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/XI2proto.h b/XI2proto.h > index 56df401..93d7e32 100644 > --- a/XI2proto.h

Re: [PATCH] dix: button state must show the logical buttons, not physical buttons

2011-12-20 Thread Chase Douglas
On 12/15/2011 06:43 PM, Peter Hutterer wrote: > If the device is mapped 3 2 1, a click on physical button 1 sends a button 3 > press, but the state was set for button 1. Fix this, the state must be set > for that button's logical mapping. > > https://bugzilla.gnome.org/show_bug.cgi?id=655928 > >

Re: [PATCH] dix: don't retrieve the syncEvents tail on an empty list

2011-12-20 Thread Chase Douglas
On 12/18/2011 09:00 PM, Peter Hutterer wrote: > An empty list points to itself but syncEvents has the list head only and is > of a different format than the elements. Thus, casting it to a QdEventPtr > gives us garbage. > > Segfaults with XTS test case Xlib13/XGrabKeyboard > > Introduced in 7af23

[PATCH edid-decode 3/3] Adjust broken header warning message

2011-12-20 Thread Tormod Volden
From: Tormod Volden Do not check edid pointer since it was already done. Remove commented return statement. Also separate declaration from code, which fixes remaining gcc -pedantic warnings. Signed-off-by: Tormod Volden --- edid-decode.c |8 1 files changed, 4 insertions(+), 4 d

[PATCH edid-decode 2/3 v2] Fix manufacture/model week/year parsing

2011-12-20 Thread Tormod Volden
From: Tormod Volden For instance, the handling of week=0xff or week=0 was not working, as could be seen from the data/sharp-* examples. Only EDID version 1.4 requires the year to be 2006 or later. Actually, data/sharp-aquos-hdmi reports 1.3 but seems to use month 0xff to flag model year, which

[PATCH edid-decode 1/3] Do the EDID version parsing before anything else

2011-12-20 Thread Tormod Volden
From: Tormod Volden So that the version conformance flags can be used in other code. Signed-off-by: Tormod Volden --- edid-decode.c | 38 +++--- 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/edid-decode.c b/edid-decode.c index ab4bde8..c3e55

[Resend: PATCH v2] Add Solaris support to DetermineClientCmd

2011-12-20 Thread Alan Coopersmith
Uses /proc/pid/psinfo to read command & partial arguments. Moves cmdsize & argsize variables into non-Solaris #else clause to avoid unused variable warnings. Fixes format mismatch errors when building with DEBUG defined on a 64-bit platform (where Mask is defined as CARD32). Signed-off-by: Alan

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Sergei Trofimovich
> On 12/20/11 12:26, Sergei Trofimovich wrote: > > What would you prefer? > > I'd prefer not supporting broken hacks pretending to be new ABI models. > > Is there any real benefit to "AMD64-x32" over simply x86-32 Whole AMD64 ISA is accessible: twice as much registers (including xmm/ymm), instru

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Sergei Trofimovich
> > On 12/20/11 08:37, Sergei Trofimovich wrote: > > > + (defined(__amd64__)&& defined(__LP64__)) /* skip x32 */ || \ > > > + defined(amd64) || \ > > > > Really? Someone defined __amd64__ on a non LP64 platform and didn't expect > > to break tons of software? Amazing... > > A bit will

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Alan Coopersmith
On 12/20/11 13:09, Sergei Trofimovich wrote: On 12/20/11 12:26, Sergei Trofimovich wrote: What would you prefer? I'd prefer not supporting broken hacks pretending to be new ABI models. Is there any real benefit to "AMD64-x32" over simply x86-32 Whole AMD64 ISA is accessible: twice as much r

Re: [Mesa-dev] [PATCH 00/20] Implement GLX_ARB_create_context and friends

2011-12-20 Thread Ian Romanick
On 12/20/2011 12:31 PM, Ian Romanick wrote: This giant series of patches implements GLX_ARB_create_context, GLX_ARB_create_context_profile, and GLX_EXT_create_context_es2_profile. One of the difficulties with this patch series is that I had to rework some of the Gallium context creation plumbing

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Alan Coopersmith
On 12/20/11 12:26, Sergei Trofimovich wrote: What would you prefer? I'd prefer not supporting broken hacks pretending to be new ABI models. Is there any real benefit to "AMD64-x32" over simply x86-32 or AMD64 for X applications? -- -Alan Coopersmith-alan.coopersm...@oracle.com

[PATCH 19/20] glx: Don't use 'new' as a function parameter name

2011-12-20 Thread Ian Romanick
From: Ian Romanick Using 'new' as a function parameter name prevents including glxclient.h the unit tests (future patch) that use the Google C++ Testing Framework. Signed-off-by: Ian Romanick --- src/glx/glxclient.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/

[PATCH 18/20] glx: Enable GLX_EXT_create_context_es2_profile

2011-12-20 Thread Ian Romanick
From: Ian Romanick This extension is only enabled if the underlying driver advertises support for OpenGL ES 2.0. This happens either through the getAPIMask function in version 2 of the DRI2 extension or implicity through version 2 of the DRISW extension. Since there is no OpenGL ES 2.0 protocol

[PATCH 17/20] dri2: Add plumbing to get context version requirements and flags to drivers

2011-12-20 Thread Ian Romanick
From: Ian Romanick This adds support for DRI_DRI2 version 3 to all of the DRI2 drivers. Signed-off-by: Ian Romanick --- .../state_trackers/dri/common/dri_context.c| 51 - .../state_trackers/dri/common/dri_context.h|4 + src/mesa/drivers/dri/common/dri_util.c

[PATCH 16/20] glx/dri2: Implement glx_screen_vtable::create_context_attribs for DRISW contexts

2011-12-20 Thread Ian Romanick
From: Ian Romanick This also enables GLX_ARB_create_context and GLX_ARB_create_context_profile if the driver supports DRI_DRISW version 3 or greater. Signed-off-by: Ian Romanick --- src/glx/drisw_glx.c | 89 ++- 1 files changed, 88 insertions(+

[PATCH 15/20] dri2: Add createContextAttribs entry point for DRISW version 3

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- include/GL/internal/dri_interface.h | 20 +++- 1 files changed, 19 insertions(+), 1 deletions(-) diff --git a/include/GL/internal/dri_interface.h b/include/GL/internal/dri_interface.h index add03c7..b21445f 100644 --- a/incl

[PATCH 14/20] glx/dri: Add utility function dri2_convert_glx_attribs

2011-12-20 Thread Ian Romanick
From: Ian Romanick This converts all of the GLX data from glXCreateContextAttribsARB to the values expected by the DRI driver interfaces. Signed-off-by: Ian Romanick --- src/glx/dri_common.c | 102 ++ src/glx/dri_common.h |6 +++ 2 files cha

[PATCH 13/20] st/mesa: Reject forward-looking contexts

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/mesa/state_tracker/st_manager.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/mesa/state_tracker/st_manager.c b/src/mesa/state_tracker/st_manager.c index 828f0d8..1cee312 100644 --- a/src/mesa/state_tr

[PATCH 12/20] dri_util: Mostly stub implementation of dri2CreateContextAttribs

2011-12-20 Thread Ian Romanick
From: Ian Romanick This adds the function and modifies dri2CreateNewContextForAPI to call it. At this point only version 2 of the DRI2 API is advertised to the loader. Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/common/dri_util.c | 97 +++ 1 files change

[PATCH 11/20] glx: Add extension strings for GLX_ARB_create_context and GLX_ARB_create_context_profile

2011-12-20 Thread Ian Romanick
From: Ian Romanick Note that these extensions are not automatically enabled for screens capable of direct-rendering. Signed-off-by: Ian Romanick --- src/glx/glxextensions.c |2 ++ src/glx/glxextensions.h |4 +++- 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/src/glx/gl

[PATCH 10/20] glx: Connect glXCreateContextAttribsARB to glXGetProcAddress

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/glx/glxcmds.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c index f7cbf2f..37c0933 100644 --- a/src/glx/glxcmds.c +++ b/src/glx/glxcmds.c @@ -2603,6 +2603,9 @@ static const

[PATCH 09/20] glx: Initial implementation of glXCreateContextAttribsARB

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/glx/Makefile |1 + src/glx/create_context.c | 117 ++ src/glx/glxclient.h |2 + src/glx/glxcmds.c|2 +- 4 files changed, 121 insertions(+), 1 deletions(-) create

[PATCH 08/20] glx/dri2: Implement glx_screen_vtable::create_context_attribs for DRI2 contexts

2011-12-20 Thread Ian Romanick
From: Ian Romanick This also enables GLX_ARB_create_context and GLX_ARB_create_context_profile if the driver supports DRI_DRI2 version 3 or greater. Signed-off-by: Ian Romanick --- src/glx/dri2_glx.c | 92 +++- 1 files changed, 91 insertions(+)

[PATCH 07/20] glx: Implement glx_screen_vtable::create_context_attribs for indirect contexts

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/glx/indirect_glx.c | 24 +++- 1 files changed, 23 insertions(+), 1 deletions(-) diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c index b628688..2399375 100644 --- a/src/glx/indirect_glx.c +++ b/src/glx/ind

[PATCH 06/20] glx: Add glx_screen_vtable::create_context_attribs

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- src/glx/dri2_glx.c |3 ++- src/glx/dri_glx.c |3 ++- src/glx/glxclient.h|8 src/glx/indirect_glx.c |3 ++- 4 files changed, 14 insertions(+), 3 deletions(-) diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_g

[PATCH 05/20] dri2: Add createContextAttribs entry point for DRI2 version 3

2011-12-20 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- include/GL/internal/dri_interface.h | 52 +- 1 files changed, 50 insertions(+), 2 deletions(-) diff --git a/include/GL/internal/dri_interface.h b/include/GL/internal/dri_interface.h index 4f768f0..add03c7 100

[PATCH 04/20] glx: Use __glX_send_client_info with XCB

2011-12-20 Thread Ian Romanick
From: Ian Romanick __glX_send_client_info only supports XCB, so use that instead of __glXClientInfo when USE_XCB is defined. Signed-off-by: Ian Romanick --- src/glx/glxext.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/glx/glxext.c b/src/glx/glxext.c index ba

[PATCH 03/20] glx: Add __glX_send_client_info super function

2011-12-20 Thread Ian Romanick
From: Ian Romanick This function picks the correct client-info protocol (based on the server's GLX version and set of extensions) and sends it to the server. Signed-off-by: Ian Romanick --- src/glx/Makefile |1 + src/glx/clientinfo.c | 156 +

[PATCH 02/20] st-api: Have context_create explain why creation failed

2011-12-20 Thread Ian Romanick
From: Ian Romanick This won't be used in the client-side libGL, but the xserver has to generate a different protocol error depending on the reason context creation failed. Signed-off-by: Ian Romanick --- src/gallium/include/state_tracker/st_api.h | 14 ++ .../state_tracke

[PATCH 01/20] st-api: Clean-up OpenGL profile handling

2011-12-20 Thread Ian Romanick
From: Ian Romanick There seems to have been two different ways to communicate the profile. There were flags and there were profiles. I've opted to remove the profile flags and use ST_PROFILE_DEFAULT (compatibility profile) and ST_PROFILE_OPENGL_CORE (core profile) consistently instead. Also ch

[PATCH 00/20] Implement GLX_ARB_create_context and friends

2011-12-20 Thread Ian Romanick
This giant series of patches implements GLX_ARB_create_context, GLX_ARB_create_context_profile, and GLX_EXT_create_context_es2_profile. One of the difficulties with this patch series is that I had to rework some of the Gallium context creation plumbing. In order for the xserver to generate the co

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Sergei Trofimovich
On Tue, 20 Dec 2011 18:02:18 +0100 (CET) Mark Kettenis wrote: > > Date: Tue, 20 Dec 2011 08:41:44 -0800 > > From: Alan Coopersmith > > > > Are you sure that every real amd64 platform that uses __amd64__ > > defines __LP64__ as well? > > Not sure if you'd call 64-bit Windows a real amd64 platfo

Re: [PATCH 00/42] Multitouch support

2011-12-20 Thread Chase Douglas
On 12/14/2011 07:01 PM, Peter Hutterer wrote: > > This is the set of patches to add multitouch support to the X server. > The lot can be found in the 'multitouch' branches of the following > repositories > > git://people.freedesktop.org/~whot/xserver > git://people.freedesktop.org/~whot/i

Re: [PATCH 37/42] Xi: add the hooks for passive touch grabs

2011-12-20 Thread Chase Douglas
On 12/14/2011 07:02 PM, Peter Hutterer wrote: > diff --git a/Xi/xipassivegrab.c b/Xi/xipassivegrab.c > index 713a165..fa2db38 100644 > --- a/Xi/xipassivegrab.c > +++ b/Xi/xipassivegrab.c > @@ -108,19 +108,29 @@ ProcXIPassiveGrabDevice(ClientPtr client) > if (stuff->grab_type != XIGrabtypeButto

Re: [PATCH util-macros 1/1] Provide a configure option to turn off the selective -Werror=... CFLAGS

2011-12-20 Thread Jeremy Huddleston
Sent from my iPhone... On Dec 20, 2011, at 5:58, Gaetan Nadon wrote: > On Mon, 2011-12-19 at 18:50 -0800, Jeremy Huddleston wrote: >> >> Provide end users with a workaround if they find code doesn't build with >> the -Werror=... flags provided by BASE_CFLAGS. This can be useful if >> newer u

Re: [PATCH 35/42] Xi: hook up touch ownership handling

2011-12-20 Thread Chase Douglas
On 12/14/2011 07:02 PM, Peter Hutterer wrote: > Ownership changes don't get processed directly when they happen, instead the > DIX submits an ownership event which goes through ProcessTouchEvents and > ProcessTouchOwnershipEvents. > Then on the required events are generated and sent to clients. >

Re: [PATCH 34/42] Xi: hook up pointer emulation for emulating touches

2011-12-20 Thread Chase Douglas
On 12/14/2011 07:02 PM, Peter Hutterer wrote: > Includes a hack for implicit grab activation, because integrating this > requires a larger rewrite and I'm not sleeping enough as it is. > Right now, we deliver the event and check before/after if there is an > implicit grab on. If one activated, then

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Alan Coopersmith
On 12/20/11 09:02, Mark Kettenis wrote: Date: Tue, 20 Dec 2011 08:41:44 -0800 From: Alan Coopersmith Are you sure that every real amd64 platform that uses __amd64__ defines __LP64__ as well? Not sure if you'd call 64-bit Windows a real amd64 platform, but given that Microsoft made the brillian

Re: [PATCH 31/42] Process and deliver touch events

2011-12-20 Thread Chase Douglas
On 12/14/2011 07:02 PM, Peter Hutterer wrote: > Does not include pointer emulation handling. > Does include partial ownership handling but not the actual processing of > ownership events. > > Note: this commit is a retroactive commit extracted from a series of ~50 > commits and may thus appear a b

Re: [PATCH libXi] Implement support for XI 2.2

2011-12-20 Thread Chase Douglas
On 12/18/2011 04:04 PM, Peter Hutterer wrote: > Adds support for the new TouchClass for multitouch-capable servers/devices. > > New events: > XITouchOwnershipEvent > > New event types handled: > XITouchBegin, XITouchUpdate, XITouchEnd > XIRawTouchBegin, XIRawTouchUpdate, XIRawTouchEnd > >

Re: [PATCH 01/21] include: add GetXI2MaskByte and GetXI2EventFilterMask to headers

2011-12-20 Thread Adam Jackson
On 12/8/11 10:36 PM, Peter Hutterer wrote: @@ -414,7 +414,7 @@ GetEventFilterMask(DeviceIntPtr dev, int evtype) return filters[dev ? dev->id : 0][evtype]; } -static inline Mask +inline Mask GetXI2EventFilterMask(int evtype) { return (1<< (evtype % 8)); This reads strangely

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Sergei Trofimovich
On Tue, 20 Dec 2011 08:41:44 -0800 Alan Coopersmith wrote: > On 12/20/11 08:37, Sergei Trofimovich wrote: > > + (defined(__amd64__)&& defined(__LP64__)) /* skip x32 */ || \ > > + defined(amd64) || \ > > Really? Someone defined __amd64__ on a non LP64 platform and didn't expect > to bre

[libXt: PATCH] amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Sergei Trofimovich
Follow changes from xproto/Xmd.h Signed-off-by: Sergei Trofimovich --- include/X11/Xtos.h |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/include/X11/Xtos.h b/include/X11/Xtos.h index 7079321..9572237 100644 --- a/include/X11/Xtos.h +++ b/include/X11/Xtos.h @@ -62,7 +6

[xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Sergei Trofimovich
Fixes build failure of libXmu: libXmu-1.1.0/work/libXmu-1.1.0/src/EditresCom.c: In function '_XEditresGetStringValues': libXmu-1.1.0/work/libXmu-1.1.0/src/EditresCom.c:2198:6: error: duplicate case value libXmu-1.1.0/work/libXmu-1.1.0/src/EditresCom.c:2194:6: error: previously used here Signed-

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Mark Kettenis
> Date: Tue, 20 Dec 2011 08:41:44 -0800 > From: Alan Coopersmith > > Are you sure that every real amd64 platform that uses __amd64__ > defines __LP64__ as well? Not sure if you'd call 64-bit Windows a real amd64 platform, but given that Microsoft made the brilliant decision to use the LLP64 mode

Re: [PATCH 27/42] dix: add helper functions for adding/removing touch listeners

2011-12-20 Thread Chase Douglas
On 12/19/2011 07:27 PM, Peter Hutterer wrote: > On Mon, Dec 19, 2011 at 07:20:38PM -0800, Chase Douglas wrote: >> On Dec 14, 2011, at 7:02 PM, Peter Hutterer wrote: >> >> [snip] >> >>> +static void >>> +TouchAddActiveGrabListener(DeviceIntPtr dev, TouchPointInfoPtr ti, >>> +

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Adam Jackson
On 12/20/11 11:41 AM, Alan Coopersmith wrote: On 12/20/11 08:37, Sergei Trofimovich wrote: + (defined(__amd64__)&& defined(__LP64__)) /* skip x32 */ || \ + defined(amd64) || \ Really? Someone defined __amd64__ on a non LP64 platform and didn't expect to break tons of software? Amazing... Yea

Re: [PATCH 4/4] miext/damage: Only wrap into the GC ops chain if there's a listener

2011-12-20 Thread Adam Jackson
On 12/17/11 8:03 PM, Aaron Plattner wrote: I think this needs to be accompanied by a change to DamageRegister (or miDamageRegister) to bump the drawable's serial number, or else creating a Damage object on a drawable won't cause GCs that are already validated against it to have the damage ops ho

Re: [xproto: PATCH] Xmd.h: amd64-x32 ABI defines sizeof(long) == sizeof (void*) == 4

2011-12-20 Thread Alan Coopersmith
On 12/20/11 08:37, Sergei Trofimovich wrote: + (defined(__amd64__)&& defined(__LP64__)) /* skip x32 */ || \ + defined(amd64) || \ Really? Someone defined __amd64__ on a non LP64 platform and didn't expect to break tons of software? Amazing... I'd expect damage from such idiocy to g

Re: [PATCH v2 23/42] dix: add touch event history helper functions

2011-12-20 Thread Chase Douglas
On 12/20/2011 07:21 AM, walter harms wrote: > > > Am 20.12.2011 04:22, schrieb Peter Hutterer: >> If touch client has not registered for ownership events and a grab above >> that client is rejected, the client needs to receive the complete event >> history. >> >> The history currently doesn't rea

Re: [PATCH] Fix memory/resource leaks in libxtrans

2011-12-20 Thread Alan Coopersmith
On 12/20/11 01:19, Alan Hourihane wrote: On 12/01/11 20:59, Alan Coopersmith wrote: On 12/01/11 04:04, Alan Hourihane wrote: Re-sending, sorry missed the [PATCH] off the subject line... Fix some resource& memory leaks in libxtrans. Signed-off-by: Alan Hourihane The free()s look just like

Re: [PATCH v2 23/42] dix: add touch event history helper functions

2011-12-20 Thread walter harms
Am 20.12.2011 04:22, schrieb Peter Hutterer: > If touch client has not registered for ownership events and a grab above > that client is rejected, the client needs to receive the complete event > history. > > The history currently doesn't really do fancy overflow handling. We assume > that the f

Re: [PATCH util-macros 1/1] Provide a configure option to turn off the selective -Werror=... CFLAGS

2011-12-20 Thread Gaetan Nadon
On Mon, 2011-12-19 at 18:50 -0800, Jeremy Huddleston wrote: > Provide end users with a workaround if they find code doesn't build with > the -Werror=... flags provided by BASE_CFLAGS. This can be useful if > newer util-macros are used with older source code that had problems, or > in the rare cas

Re: [PATCH] Fix memory/resource leaks in libxtrans

2011-12-20 Thread Alan Hourihane
On 12/01/11 20:59, Alan Coopersmith wrote: > On 12/01/11 04:04, Alan Hourihane wrote: >> Re-sending, sorry missed the [PATCH] off the subject line... >> >> Fix some resource& memory leaks in libxtrans. >> >> Signed-off-by: Alan Hourihane > > The free()s look just like a patch I wrote yesterday to

Re: [PULL] XQuartz and kdrive build fixes

2011-12-20 Thread Keith Packard
On Mon, 19 Dec 2011 23:29:52 -0800, Jeremy Huddleston wrote: > Jeremy Huddleston (3): > XQuartz: Provide in-tree implementation of strndup(3) if needed > configure.ac: Make Xephyr dependency error message more informative > kdrive/linux: Fix compilation with -Werror=int-to-pointer