<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On 19 August 2014 11:25, Giulio Camuffo <span dir="ltr"><<a href="mailto:giuliocamuffo@gmail.com" target="_blank">giuliocamuffo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This adds a function weston_keyboard_set_leds() which can be used<br>
to change the state of the num lock and the caps lock leds.<br>
Only the evdev backend supports this, since it doesn't make sense<br>
for embedded sessions.<br></blockquote><div><br></div><div>This is missing the bit for actual evdev support, as well as a user. Please also move the variable declarations to the top of the function. The leds/active naming is a bit confusing too; something about mask/value would be nice, as well as getting rid of the complex ternary statements.</div>
<div><br></div><div>Other than that, looks good to me, thanks.</div><div><br></div><div>Cheers,</div><div>Daniel </div></div></div></div>