OSDN Git Service

ui: remove bogus call to reset_keys() in vnc_init_state
authorDaniel P. Berrange <berrange@redhat.com>
Thu, 29 Sep 2016 15:45:38 +0000 (16:45 +0100)
committerGerd Hoffmann <kraxel@redhat.com>
Thu, 13 Oct 2016 07:22:30 +0000 (09:22 +0200)
commit2df2041036ee63ff9116631c6214e3ffb5f4bf45
tree3b7846d310fd480562ab4ad23b0a5ded885affe6
parentf54195ddf7460df077bec42a2475fd2d30a3f3f1
ui: remove bogus call to reset_keys() in vnc_init_state

The vnc_init_state method calls reset_keys() to reset the
modifier key state. This was originally added in

  commit 53762ddb277c690e486d0e17b10591774248c8cf
  Author: malc <malc@c046a42c-6fe2-441c-8c8c-71466251a162>
  Date:   Mon Dec 1 20:57:52 2008 +0000

    Reset the key modifiers upon client connect

This was valid at this time because there was only the
single VncState object which was persistent across client
connections and so needed resetting.

The persistent data was later split off into VncDisplay
and VncState was allocated at time of client connection:

  commit 753b4053311ff1437d99726970b1e7e6bf38249b
  Author: aliguori <aliguori@c046a42c-6fe2-441c-8c8c-71466251a162>
  Date:   Mon Feb 16 14:59:30 2009 +0000

    Support multiple VNC clients (Brian Kress)

at which point the modifier state is always 0 due to
use of g_new0. As such the reset_keys() call has been
a no-op ever since.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Message-id: 1475163940-26094-8-git-send-email-berrange@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
ui/vnc.c