[PATCH kdrive/evdev] kdrive/evdev: update keyboard LEDs (#22302)

Adam Jackson ajax at nwnk.net
Mon Feb 22 18:48:28 UTC 2016


On Mon, 2016-02-15 at 11:32 -0200, Laércio de Sousa wrote:

> Implement missing parts in kdrive evdev driver for
> correct update of evdev keyboard LEDs.

Hah! I like how this was there and just not wired up.

> +    if (!ke) {
> +        ErrorF("Can't update LEDs of disabled evdev keyboard %s.\n", ki->path);
> +        return;
> +    }

I'm not a fan of error messages for routine conditions. If this is
really a "shouldn't happen" case then fine, but it looks like this will
emit a message every time you have >=1 disabled keyboards and hit
CapsLock on an enabled keyboard. If I'm reading that correctly, can we
just drop this message?

- ajax


More information about the xorg-devel mailing list