[PATCH xserver] xwayland: add "tablet" into the tablet device names

Olivier Fourdan fourdan at gmail.com
Mon Jun 11 09:54:29 UTC 2018


On Mon, 11 Jun 2018 at 11:40, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> On Mon, Jun 11, 2018 at 11:21:25AM +0200, Olivier Fourdan wrote:
[...]
> > > -    pad->xdevice = add_device(pad->seat, "xwayland-pad",
> > > +    pad->xdevice = add_device(pad->seat, "xwayland-tablet-pad",
> >
> > Previous ones used a space, here you use a dash between
> > “xwayland-tablet” and the device type, I'd rather have that
> > consistent.
>
> at the risk of disagreeing over the optical appearance of a velocipede
> storage environment: the xwayland-tablet-pad refers to an interface in the
> same way as xwayland-tablet does. The space-separated part is the tool which
> we only have in the tablet interface, not the tablet-pad one.

Oh, I see! There is a logic behind that apparent inconsistency! :)

> Not that any of this really matters for the user who sees this :)

Indeed, so either way, my R-b remains :)

Cheers,
Olivier


More information about the wayland-devel mailing list