[PATCH libinput] gestures: reduce 2fg scroll timeout to 500ms

2015-08-02 Thread Peter Hutterer
Holding the fingers in place without moving for 500ms is long enough to lock in a scroll gesture, especially while we're still waiting for the rest of the stack to expose pinch gestures. Signed-off-by: Peter Hutterer --- src/evdev-mt-touchpad-gestures.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH libinput] touchpad: always enable the bottom-most area for thumb detection

2015-08-02 Thread Peter Hutterer
If the touchpad is higher than 50mm, enable bottom area thumb detection. This only applies to the bottom-most 8mm and only if the touch remains unmoving in that area. Signed-off-by: Peter Hutterer --- src/evdev-mt-touchpad.c | 36 1 file changed, 20 insertion

Re: [PATCH v2 libinput] filter: move the pointer acceleration profiles back to units/ms

2015-08-02 Thread Jonas Ådahl
On Mon, Aug 03, 2015 at 10:27:01AM +1000, Peter Hutterer wrote: > There is no need here to use µs since we're just handling speeds/thresholds, > not actual events where a ms granularity can be too high. > > Moving back to ms lets us drop a bunch of zeroes that clutter up the code, and > since the

[PATCH v2 libinput] filter: move the pointer acceleration profiles back to units/ms

2015-08-02 Thread Peter Hutterer
There is no need here to use µs since we're just handling speeds/thresholds, not actual events where a ms granularity can be too high. Moving back to ms lets us drop a bunch of zeroes that clutter up the code, and since the acceleration functions are a bit magic anyway, having the various 1000.0 f

Re: [PATCH] [RDP compositor] don't crash when the key file doesn't exist

2015-08-02 Thread Dawid Gajownik
On Sat, Aug 1, 2015 at 7:35 PM, David FORT wrote: > This allows to close the remaining issue in > https://bugs.freedesktop.org/show_bug.cgi?id=91390 I'm a newbie here so I don't think I can make reviews but the patch works fine for me. [12:56:36.661] Loading module '/home/user/usr/lib/weston/des

Re: [PATCH weston] compositor-rdp: free() can handle NULL pointers

2015-08-02 Thread Bryce Harrington
On Sat, Aug 01, 2015 at 10:03:12PM +0200, Hardening wrote: > Le 01/08/2015 16:02, Dawid Gajownik a écrit : > > There's no need to check if a pointer exists before passing it to free(). > > free() can handle NULL pointers. > > > > Signed-off-by: Dawid Gajownik > > Reviewed-By: David FORT Thanks