[PATCH] xwayland: Only disable/enable devices on capabilities change

Daniel Stone daniel at fooishbar.org
Tue Jul 22 09:11:53 PDT 2014


Hi,

On 22 July 2014 16:55, Carlos Garnacho <carlosg at gnome.org> wrote:

> Anytime a capability is first reported, the device is created, but after
> that, it is only disabled/enabled.
>
> This is a closer behavior to what Xorg does on VT switch, at the expense
> of maybe leaving a dangling "physical" device if a capability goes for
> good.
> Otherwise, any DeviceIntPtr (re)created after server initialization will be
> left floating, and bad things happen when the wayland enter event handler
> tries to update cursor position based on a floating device.
>
> Signed-off-by: Carlos Garnacho <carlosg at gnome.org>
>

A little confusing at first, but looks OK to me.

Reviewed-by: Daniel Stone <daniel at fooishbar.org>

Cheers,
Daniel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140722/8d0ac351/attachment.html>


More information about the xorg-devel mailing list