Re: [PATCH] xwayland: Avoid repeatedly looping through window ancestor chain

2017-08-11 Thread Roman Gilg
On Thu, Aug 10, 2017 at 2:49 PM, Pekka Paalanen wrote: > > how does this work when windows are reparented? Does reparenting force > them to go through the unrealize/realize steps again? > I assumed this because it is described this way for the XLib function XReparentWindow:

[PATCH] xwayland: Remove unneeded double test on confined_pointer

2017-08-11 Thread Roman Gilg
In xwl_seat_maybe_lock_on_hidden_cursor the confined_pointer is destroyed in case it exists. We can remove this since we do it in the subsequent call to xwl_seat_create_pointer_warp_emulator as well. Signed-off-by: Roman Gilg --- hw/xwayland/xwayland-input.c | 3 --- 1 file

Re: [PATCH xserver] meson: Fix epoll detection

2017-08-11 Thread Peter Harris
On 2017-08-11 5:50 AM, Peter Hutterer wrote: > On Tue, Aug 08, 2017 at 11:16:13AM -0400, Peter Harris wrote: >> The epoll code depends on epoll_create1, not epoll_create. >> >> The trinary " ? 1 : false" is used because HAVE_EPOLL_CREATE1 is tested >> with #if instead of #ifdef. > > might be

Re: [PATCH xserver 2/2] travis: Request a new docker image with xkb-data and xkbcomp installed.

2017-08-11 Thread Adam Jackson
On Fri, 2017-08-04 at 13:19 -0700, Eric Anholt wrote: > Xvfb was erroring out with: > > XKB: Failed to compile keymap > Keyboard initialization failed. This could be a missing or incorrect > setup of xkeyboard-config. > (EE) Fatal server error: > (EE) Failed to activate virtual core keyboard:

Re: Second Feedback request for my GSoC project to improve Present support in Xwayland

2017-08-11 Thread Pekka Paalanen
On Tue, 1 Aug 2017 20:53:02 +0200 Roman Gilg wrote: > > Summary: > > > (1) Deactivate queued presentation support for now (until we know which > kind of Wayland protocol to use for). > (2) Per window flipping when the pixmap flipping window region equals the

Re: [PATCH xserver] meson: Fix epoll detection

2017-08-11 Thread Peter Hutterer
On Tue, Aug 08, 2017 at 11:16:13AM -0400, Peter Harris wrote: > The epoll code depends on epoll_create1, not epoll_create. > > The trinary " ? 1 : false" is used because HAVE_EPOLL_CREATE1 is tested > with #if instead of #ifdef. > > Signed-off-by: Peter Harris > --- >