All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrange" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: Gerd Hoffmann <kraxel@redhat.com>,
	"Daniel P. Berrange" <berrange@redhat.com>
Subject: [Qemu-devel] [PATCH v6 4/4] ui: ignore hardware keycode 255 on win32
Date: Tue, 12 Dec 2017 14:04:59 +0000	[thread overview]
Message-ID: <20171212140459.2234-5-berrange@redhat.com> (raw)
In-Reply-To: <20171212140459.2234-1-berrange@redhat.com>

It is a reserved value and doesn't have a corresponding
valid scancode.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
---
 ui/gtk.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/ui/gtk.c b/ui/gtk.c
index 89cc81b708..687560b963 100644
--- a/ui/gtk.c
+++ b/ui/gtk.c
@@ -1206,6 +1206,12 @@ static gboolean gd_key_event(GtkWidget *widget, GdkEventKey *key, void *opaque)
         return TRUE;
     }
 
+#ifdef WIN32
+    /* on windows, we ought to ignore the reserved key event? */
+    if (key->hardware_keycode == 0xff)
+        return false;
+#endif
+
     if (key->keyval == GDK_KEY_Pause) {
 #ifdef G_OS_WIN32
         /* for some reason GDK does not fill keyval for VK_PAUSE
-- 
2.14.3

  parent reply	other threads:[~2017-12-12 14:05 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-12 14:04 [Qemu-devel] [PATCH v6 0/4] Convert frontends to use keycodemapdb Daniel P. Berrange
2017-12-12 14:04 ` [Qemu-devel] [PATCH v6 1/4] ui: convert the SDL2 frontend to keycodemapdb Daniel P. Berrange
2017-12-12 14:04 ` [Qemu-devel] [PATCH v6 2/4] ui: convert GTK and SDL1 frontends " Daniel P. Berrange
2017-12-12 14:04 ` [Qemu-devel] [PATCH v6 3/4] ui: add fix for GTK Pause key handling on Win32 Daniel P. Berrange
2017-12-12 14:04 ` Daniel P. Berrange [this message]
2017-12-12 14:50 ` [Qemu-devel] [PATCH v6 0/4] Convert frontends to use keycodemapdb no-reply
2018-01-15 15:10 ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171212140459.2234-5-berrange@redhat.com \
    --to=berrange@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.