<div dir="ltr">Ah, yes! No problem! We can drop it.<br></div><div class="gmail_extra"><br><div class="gmail_quote">2016-02-22 15:48 GMT-03:00 Adam Jackson <span dir="ltr"><<a href="mailto:ajax@nwnk.net" target="_blank">ajax@nwnk.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, 2016-02-15 at 11:32 -0200, Laércio de Sousa wrote:<br>
<br>
> Implement missing parts in kdrive evdev driver for<br>
> correct update of evdev keyboard LEDs.<br>
<br>
</span>Hah! I like how this was there and just not wired up.<br>
<span class=""><br>
> +    if (!ke) {<br>
> +        ErrorF("Can't update LEDs of disabled evdev keyboard %s.\n", ki->path);<br>
> +        return;<br>
> +    }<br>
<br>
</span>I'm not a fan of error messages for routine conditions. If this is<br>
really a "shouldn't happen" case then fine, but it looks like this will<br>
emit a message every time you have >=1 disabled keyboards and hit<br>
CapsLock on an enabled keyboard. If I'm reading that correctly, can we<br>
just drop this message?<br>
<br>
- ajax<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div><b>Laércio de Sousa</b><br></div><div style="text-align:center"><div style="text-align:left"><i>Orientador de Informática</i></div><div style="text-align:left"><u>Escola Municipal "Professor Eulálio Gruppi"</u></div><div style="text-align:left"><i>Rua Ismael da Silva Mello, 559, Mogi Moderno</i></div><i><div style="text-align:left"><i>Mogi das Cruzes - SP</i></div><div style="text-align:left"><i>CEP 08717-390</i></div></i></div><div><span style="font-family:arial;font-size:small">Telefone: (11) 4726-8313</span></div></div></div>
</div>