<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 9 December 2014 at 21:36, Derek Foreman <span dir="ltr"><<a href="mailto:derekf@osg.samsung.com" target="_blank">derekf@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Distill has 2 ls...<br>
<span class=""><br>
On 05/12/14 07:36 AM, Marek Chalupa wrote:<br>
> Split out the real code from event handlers (like pointer_handle_motion,<br>
> etc.) and put it into standalone function. We'll use it in follow-up<br>
> patch where we will register handlers for toytoolkit clients and we'll<br>
> use the new functions so that the code won't be duplicated.<br>
><br>
> Signed-off-by: Marek Chalupa <<a href="mailto:mchqwerty@gmail.com">mchqwerty@gmail.com</a>><br>
> ---<br>
>  tests/weston-test-client-helper.c | 170 +++++++++++++++++++++++++++++---------<br>
>  1 file changed, 130 insertions(+), 40 deletions(-)<br>
><br>
</span>...<br>
<span class=""><br>
> +<br>
> +static void<br>
> +store_keyboard_modifiers(struct keyboard *keyboard, uint32_t mods_depressed,<br>
> +                       uint32_t mods_latched, uint32_t mods_locked,<br>
> +                       uint32_t group)<br>
> +{<br>
>       keyboard->mods_depressed = mods_depressed;<br>
>       keyboard->mods_latched = mods_latched;<br>
>       keyboard->mods_locked = mods_locked;<br>
>       keyboard->group = group;<br>
><br>
> -     fprintf(stderr, "test-client: got keyboard modifiers %u %u %u %u\n",<br>
> +     fprintf(stderr, "test-client: got keyboard modifiers %x %x %x %x\n",<br>
>               mods_depressed, mods_latched, mods_locked, group);<br>
>  }<br>
<br>
</span>I personally prefer 0x%x to %x on the off chance something comes up that<br>
could be decimal.  It can be confusing.<br>
<br>
Is this change an important part of the patch?<br></blockquote><div><br></div><div>Not at all, I don't even remember I changed this :D I probably made it when I was debugging something.<br>Nevermind.., I'll revert it back to %u. Or is 0x%x better? <br><br></div><div>Thanks,<br>Marek<br></div></div><br></div></div>