Another wl_display_read_events bug

Hi, I discovered another bug in wl_display_read_events(). In this series I'm adding a test and fix for it. Hopefully, these are the last patches regarding threading from me atm (except for the promised documentation). /* cosmetic changes and code reuse */ [PATCH 1/4] client: add

[PATCH 1/4] client: add display_wakeup_threads function

This helper function wraps the always-repeated pattern: display-read_serial++; pthread_cond_broadcast(display-reader_cond); Signed-off-by: Marek Chalupa mchqwe...@gmail.com --- src/wayland-client.c | 37 + 1 file changed, 25 insertions(+), 12 deletions(-)

[PATCH 2/4] display-test: make use of create_thread function

This function is used in one test only, but its functionality can be used in another tests to (create thread and wait until it is sleeping). We just need to pass the starting function for the thread as an argument. Signed-off-by: Marek Chalupa mchqwe...@gmail.com --- tests/display-test.c | 55

Re: [PATCH 2/2] client: wake up sleeping threads on all return paths from read_events

This patch is now obsolete, see http://lists.freedesktop.org/archives/wayland-devel/2014-August/016966.html On 28 August 2014 12:20, Marek Chalupa mchqwe...@gmail.com wrote: Hmm, thinking about that... Maybe this is not the right solution. Programmer should probably do something like: do {

Re: [PATCH libinput 05/11] evdev: apply calibration to multitouch values as well

Hi, On 08/29/2014 01:50 AM, Peter Hutterer wrote: On Thu, Aug 28, 2014 at 03:02:33PM +0200, Hans de Goede wrote: Hi, On 08/27/2014 06:31 AM, Peter Hutterer wrote: We apply calibration to single-touch and absolute devices, but we might as well do so for multitouch events. Signed-off-by:

Re: [PATCH libinput 06/11] util: add a couple of 3x3 matrix helper functions

Hi, On 08/29/2014 01:51 AM, Peter Hutterer wrote: On Thu, Aug 28, 2014 at 03:04:30PM +0200, Hans de Goede wrote: Hi, On 08/27/2014 06:31 AM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/libinput-util.h | 100

Re: [PATCH libinput 07/11] evdev: switch to a normalized transformation matrix

Hi, On 08/29/2014 02:04 AM, Peter Hutterer wrote: On Thu, Aug 28, 2014 at 03:07:26PM +0200, Hans de Goede wrote: Hi, On 08/27/2014 06:31 AM, Peter Hutterer wrote: The big change here is the requirement to have the translation component in a device-normalized coordinate space. Without

[PATCH weston] xwm: Check whether the seat is NULL sometimes in weston_wm_handle_button

Under some certain circumstances, pointer button may have been released when frame is still being resized/moved. When this happens, the picked seat is NULL and it will segfault when moving/resizing surfaces. Check whether the seat is NULL and ignore move/resize in that case. Fixes:

[PATCH] pixman-backend: add support for zooming

Currently if you try to zoom with mod+scrollwheel the pixman backend will stop rendering anything at all and will continuously log pixman renderer does not support zoom, giving the impression that the server is hung. Instead, this patch adds the missing zoom functionality. This should close BUG

Self introduction

Hello All, As usual, a small self introduction... My name is Matthieu Gautier, French developer of 30 years old. I'm a developer for years now and I've worked on a lot of projects related to embedded and graphics (video game, scientific visualization and digital TV). My last job was to work

[PATCH] touch-input: Don't dereference NULL pointer during full screen fade

If a full screen program is fading out and a touch start happens, it will result in a NULL pointer dereference when weston_touch_set_focus tries to derefernce view-surface-resource. Instead, this patch sets the focus to NULL, which should be the same as if the program was destroyed during the

Re: [PATCH] touch-input: Don't dereference NULL pointer during full screen fade

I should mention I don't have a touch capable screen and am only doing partial testing with a modified version of libinput. More testing would be nice. :) On 29/08/14 01:12 PM, Derek Foreman wrote: If a full screen program is fading out and a touch start happens, it will result in a NULL

Re: [PATCH] pixman-backend: add support for zooming

Derek, I haven't had a chance to look that hard at or play with this patch yet. Hopefully I can look at it before too long. One quick question though: Have you verified that this works with all of the output transforms? I'm sorry if this seems premature but a) pixman renderer stuff is hard to

Re: Self introduction

Welcome Matthieu, On Fri, Aug 29, 2014 at 8:40 AM, Matthieu Gautier d...@mgautier.fr wrote: Hello All, As usual, a small self introduction... I don't know that it's really usual on this list, but meh. My name is Matthieu Gautier, French developer of 30 years old. I'm a developer for

[PATCH weston] xwm: Do not activate override redirect windows

We shouldn't do WM-y things on an O-R window, including setting input focus to it. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=81273 Signed-off-by: Boyan Ding stu_...@126.com --- xwayland/window-manager.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/xwayland/window-manager.c