[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vnc: Make ledstate comparison before modifiers
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH] vnc: Make ledstate comparison before modifiers updated |
Date: |
Wed, 15 May 2013 10:31:05 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 |
On 05/15/13 10:20, Lei Li wrote:
> The ledstate should be compared before modifiers updated,
> otherwise the ledstate would be the same as current_led_state.
>
> Reported-by: Gerd Hoffmann <address@hidden>
> Signed-off-by: Lei Li <address@hidden>
> ---
> ui/vnc.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/ui/vnc.c b/ui/vnc.c
> index 89108de..dfc7459 100644
> --- a/ui/vnc.c
> +++ b/ui/vnc.c
> @@ -1601,6 +1601,7 @@ static void kbd_leds(void *opaque, int ledstate)
> {
> VncState *vs = opaque;
> int caps, num, scr;
> + bool has_changed = (ledstate != current_led_state(vs));
>
> caps = ledstate & QEMU_CAPS_LOCK_LED ? 1 : 0;
> num = ledstate & QEMU_NUM_LOCK_LED ? 1 : 0;
> @@ -1617,7 +1618,7 @@ static void kbd_leds(void *opaque, int ledstate)
> }
>
> /* Sending the current led state message to the client */
> - if (ledstate != current_led_state(vs)) {
> + if (has_changed) {
> vnc_led_state_change(vs);
> }
> }
>
Looks good to me.
Reviewed-by: Laszlo Ersek <address@hidden>