[PATCH evdev] Fix relative events with swapped axes

2011-12-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com After we swap the axes, we only call valuator_mask_set for axes that are not zero, so we need to unset the axes that became zero when swapped. Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- src/evdev.c |4 1 files changed, 4

Re: [PATCH evdev] Fix absolute events with swapped axes

2011-12-15 Thread Paulo Zanoni
Hi 2011/12/14 Chase Douglas chase.doug...@canonical.com: Would it be better to remove the evdev axis swap property? If we still want it around, it would probably be easier to have it set the transformation matrix under the covers. It would make the transformation matrix and evdev axis

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

2011-12-15 Thread Fernando Carrijo
Peter Hutterer peter.hutte...@who-t.net wrote: +static void +TouchAddActiveGrabListener(DeviceIntPtr dev, TouchPointInfoPtr ti, + InternalEvent *ev, GrabPtr grab) +{ +if (!ti-emulate_pointer +(grab-grabtype == CORE || grab-grabtype == XI)) +

[PATCH:inputproto] specs: Fix tiny typo.

2011-12-15 Thread Cyril Brulebois
Signed-off-by: Cyril Brulebois k...@debian.org --- specs/XI2proto.txt |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/specs/XI2proto.txt b/specs/XI2proto.txt index 8b79210..46895d8 100644 --- a/specs/XI2proto.txt +++ b/specs/XI2proto.txt @@ -2390,7 +2390,7 @@ require the

Re: [PATCH 40/42] dix: Remove touch grabs if the grab disappears

2011-12-15 Thread Fernando Carrijo
Peter Hutterer peter.hutte...@who-t.net wrote: +void +TouchListenerGone(XID resource) +{ +TouchPointInfoPtr ti; +DeviceIntPtr dev; +InternalEvent *events = InitEventList(GetMaximumEventsNum()); Forgot to deallocate the event list. +int i, j, k, nev; + +if (!events)

Re: [PULL] Input changes in preparation for multitouch

2011-12-15 Thread Aaron Plattner
On 12/08/2011 07:04 PM, Keith Packard wrote: * PGP Signed by an unknown key On Fri, 9 Dec 2011 10:50:33 +1000, Peter Huttererpeter.hutte...@who-t.net wrote: dix: switch the syncEvent queue to a struct list This exposes PlayReleasedEvents as a global function (for testing),

[PATCH] make Xephyr resizeable

2011-12-15 Thread Adam Jackson
From: Vic Lee ll...@163.com Modified from Vic's original patch to make Xephyr always be resizeable, instead of only if embedded in something else. https://bugs.freedesktop.org/show_bug.cgi?id=25804 Reviewed-by: Adam Jackson a...@redhat.com Signed-off-by: Vic Lee ll...@163.com ---

Re: [PATCH xserver 2/4] dmx: fix distcheck failure, missing compsize.h in Makefile.am

2011-12-15 Thread Keith Packard
On Fri, 09 Dec 2011 07:45:02 -0500, Gaetan Nadon mems...@videotron.ca wrote: which was added in commit: dmx: Build fix for -Werror=implicit-function-declaration Merged. f7ea7a3..2c1d0a5 master - master -- keith.pack...@intel.com pgphpRmdfVa1g.pgp Description: PGP signature

Re: [PATCH:inputproto] specs: Fix tiny typo.

2011-12-15 Thread Peter Hutterer
On Thu, Dec 15, 2011 at 05:07:54PM +0100, Cyril Brulebois wrote: Signed-off-by: Cyril Brulebois k...@debian.org --- specs/XI2proto.txt |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/specs/XI2proto.txt b/specs/XI2proto.txt index 8b79210..46895d8 100644 ---

[PATCH v2 02/42] dix: split positionSprite into scale_to_desktop and positionSprite

2011-12-15 Thread Peter Hutterer
For future touch points, we need positionSprite to calculate the coordinates but we don't want to actually change the cursor position for non-emulating touches. No functional changes at this point. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Changes to v1: - instead of an update

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

2011-12-15 Thread Peter Hutterer
On Thu, Dec 15, 2011 at 01:40:48PM -0200, Fernando Carrijo wrote: Peter Hutterer peter.hutte...@who-t.net wrote: +static void +TouchAddActiveGrabListener(DeviceIntPtr dev, TouchPointInfoPtr ti, + InternalEvent *ev, GrabPtr grab) +{ +if

Re: [PATCH 40/42] dix: Remove touch grabs if the grab disappears

2011-12-15 Thread Peter Hutterer
On Thu, Dec 15, 2011 at 03:41:39PM -0200, Fernando Carrijo wrote: Peter Hutterer peter.hutte...@who-t.net wrote: +void +TouchListenerGone(XID resource) +{ +TouchPointInfoPtr ti; +DeviceIntPtr dev; +InternalEvent *events = InitEventList(GetMaximumEventsNum()); Forgot

Re: [PATCH v2 02/42] dix: split positionSprite into scale_to_desktop and positionSprite

2011-12-15 Thread Jamey Sharp
Welp, I'm happy! Reviewed-by: Jamey Sharp ja...@minilop.net On Fri, Dec 16, 2011 at 10:28:31AM +1000, Peter Hutterer wrote: For future touch points, we need positionSprite to calculate the coordinates but we don't want to actually change the cursor position for non-emulating touches. No

Re: [PATCH libX11] XQueryColors: Split a request into multiple requests if necessary

2011-12-15 Thread James Cloos
I pushed this as commit 87e10a7b9a97c951ab4d477f6119ac0a6a66. One of the comments in the bugz Jamey suggested that an async reply handler, but given that such a patch hasn't appeared in the half-decade since he opened bugz 9120, it seems reasonable to push the fix we have until and unless

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

2011-12-15 Thread Peter Hutterer
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 Signed-off-by: Peter Hutterer peter.hutte...@who-t.net ---

Re: [PATCH evdev] Fix relative events with swapped axes

2011-12-15 Thread Peter Hutterer
On Thu, Dec 15, 2011 at 12:26:37PM -0200, przan...@gmail.com wrote: From: Paulo Zanoni paulo.r.zan...@intel.com After we swap the axes, we only call valuator_mask_set for axes that are not zero, so we need to unset the axes that became zero when swapped. Signed-off-by: Paulo Zanoni

Re: [PATCH evdev] Fix absolute events with swapped axes

2011-12-15 Thread Peter Hutterer
On Thu, Dec 15, 2011 at 12:33:51PM -0200, Paulo Zanoni wrote: Hi 2011/12/14 Chase Douglas chase.doug...@canonical.com: Would it be better to remove the evdev axis swap property? If we still want it around, it would probably be easier to have it set the transformation matrix under the