kbd-state: use state tracker for sdl2
Use the new keyboard state tracked for sdl2. We can drop the modifier state tracking from sdl2. Also keyup code is simpler, the state tracker will take care to not send suspious keyup events to the guest. Signed-off-by: Gerd Hoffmann <kraxel@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Message-id: 20190122092814.14919-4-kraxel@redhat.com
This commit is contained in:
parent
85b03694e1
commit
07333e1ca3
@ -10,6 +10,7 @@
|
|||||||
# include <SDL_image.h>
|
# include <SDL_image.h>
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
#include "ui/kbd-state.h"
|
||||||
#ifdef CONFIG_OPENGL
|
#ifdef CONFIG_OPENGL
|
||||||
# include "ui/egl-helpers.h"
|
# include "ui/egl-helpers.h"
|
||||||
#endif
|
#endif
|
||||||
@ -30,6 +31,7 @@ struct sdl2_console {
|
|||||||
int idle_counter;
|
int idle_counter;
|
||||||
int ignore_hotkeys;
|
int ignore_hotkeys;
|
||||||
SDL_GLContext winctx;
|
SDL_GLContext winctx;
|
||||||
|
QKbdState *kbd;
|
||||||
#ifdef CONFIG_OPENGL
|
#ifdef CONFIG_OPENGL
|
||||||
QemuGLShader *gls;
|
QemuGLShader *gls;
|
||||||
egl_fb guest_fb;
|
egl_fb guest_fb;
|
||||||
|
@ -30,8 +30,6 @@
|
|||||||
#include "ui/sdl2.h"
|
#include "ui/sdl2.h"
|
||||||
#include "sysemu/sysemu.h"
|
#include "sysemu/sysemu.h"
|
||||||
|
|
||||||
static uint8_t modifiers_state[SDL_NUM_SCANCODES];
|
|
||||||
|
|
||||||
void sdl2_process_key(struct sdl2_console *scon,
|
void sdl2_process_key(struct sdl2_console *scon,
|
||||||
SDL_KeyboardEvent *ev)
|
SDL_KeyboardEvent *ev)
|
||||||
{
|
{
|
||||||
@ -43,31 +41,10 @@ void sdl2_process_key(struct sdl2_console *scon,
|
|||||||
}
|
}
|
||||||
|
|
||||||
qcode = qemu_input_map_usb_to_qcode[ev->keysym.scancode];
|
qcode = qemu_input_map_usb_to_qcode[ev->keysym.scancode];
|
||||||
|
qkbd_state_key_event(scon->kbd, qcode, ev->type == SDL_KEYDOWN);
|
||||||
/* modifier state tracking */
|
|
||||||
switch (ev->keysym.scancode) {
|
|
||||||
case SDL_SCANCODE_LCTRL:
|
|
||||||
case SDL_SCANCODE_LSHIFT:
|
|
||||||
case SDL_SCANCODE_LALT:
|
|
||||||
case SDL_SCANCODE_LGUI:
|
|
||||||
case SDL_SCANCODE_RCTRL:
|
|
||||||
case SDL_SCANCODE_RSHIFT:
|
|
||||||
case SDL_SCANCODE_RALT:
|
|
||||||
case SDL_SCANCODE_RGUI:
|
|
||||||
if (ev->type == SDL_KEYUP) {
|
|
||||||
modifiers_state[ev->keysym.scancode] = 0;
|
|
||||||
} else {
|
|
||||||
modifiers_state[ev->keysym.scancode] = 1;
|
|
||||||
}
|
|
||||||
break;
|
|
||||||
default:
|
|
||||||
/* nothing */
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!qemu_console_is_graphic(con)) {
|
if (!qemu_console_is_graphic(con)) {
|
||||||
bool ctrl = (modifiers_state[SDL_SCANCODE_LCTRL] ||
|
bool ctrl = qkbd_state_modifier_get(scon->kbd, QKBD_MOD_CTRL);
|
||||||
modifiers_state[SDL_SCANCODE_RCTRL]);
|
|
||||||
if (ev->type == SDL_KEYDOWN) {
|
if (ev->type == SDL_KEYDOWN) {
|
||||||
switch (ev->keysym.scancode) {
|
switch (ev->keysym.scancode) {
|
||||||
case SDL_SCANCODE_RETURN:
|
case SDL_SCANCODE_RETURN:
|
||||||
|
12
ui/sdl2.c
12
ui/sdl2.c
@ -38,7 +38,6 @@ static int gui_grab; /* if true, all keyboard/mouse events are grabbed */
|
|||||||
|
|
||||||
static int gui_saved_grab;
|
static int gui_saved_grab;
|
||||||
static int gui_fullscreen;
|
static int gui_fullscreen;
|
||||||
static int gui_keysym;
|
|
||||||
static int gui_grab_code = KMOD_LALT | KMOD_LCTRL;
|
static int gui_grab_code = KMOD_LALT | KMOD_LCTRL;
|
||||||
static SDL_Cursor *sdl_cursor_normal;
|
static SDL_Cursor *sdl_cursor_normal;
|
||||||
static SDL_Cursor *sdl_cursor_hidden;
|
static SDL_Cursor *sdl_cursor_hidden;
|
||||||
@ -330,6 +329,7 @@ static void handle_keydown(SDL_Event *ev)
|
|||||||
int win;
|
int win;
|
||||||
struct sdl2_console *scon = get_scon_from_window(ev->key.windowID);
|
struct sdl2_console *scon = get_scon_from_window(ev->key.windowID);
|
||||||
int gui_key_modifier_pressed = get_mod_state();
|
int gui_key_modifier_pressed = get_mod_state();
|
||||||
|
int gui_keysym = 0;
|
||||||
|
|
||||||
if (!scon->ignore_hotkeys && gui_key_modifier_pressed && !ev->key.repeat) {
|
if (!scon->ignore_hotkeys && gui_key_modifier_pressed && !ev->key.repeat) {
|
||||||
switch (ev->key.keysym.scancode) {
|
switch (ev->key.keysym.scancode) {
|
||||||
@ -410,16 +410,9 @@ static void handle_keydown(SDL_Event *ev)
|
|||||||
static void handle_keyup(SDL_Event *ev)
|
static void handle_keyup(SDL_Event *ev)
|
||||||
{
|
{
|
||||||
struct sdl2_console *scon = get_scon_from_window(ev->key.windowID);
|
struct sdl2_console *scon = get_scon_from_window(ev->key.windowID);
|
||||||
int gui_key_modifier_pressed = get_mod_state();
|
|
||||||
|
|
||||||
scon->ignore_hotkeys = false;
|
scon->ignore_hotkeys = false;
|
||||||
|
sdl2_process_key(scon, &ev->key);
|
||||||
if (!gui_key_modifier_pressed) {
|
|
||||||
gui_keysym = 0;
|
|
||||||
}
|
|
||||||
if (!gui_keysym) {
|
|
||||||
sdl2_process_key(scon, &ev->key);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void handle_textinput(SDL_Event *ev)
|
static void handle_textinput(SDL_Event *ev)
|
||||||
@ -823,6 +816,7 @@ static void sdl2_display_init(DisplayState *ds, DisplayOptions *o)
|
|||||||
sdl2_console[i].dcl.ops = &dcl_2d_ops;
|
sdl2_console[i].dcl.ops = &dcl_2d_ops;
|
||||||
#endif
|
#endif
|
||||||
sdl2_console[i].dcl.con = con;
|
sdl2_console[i].dcl.con = con;
|
||||||
|
sdl2_console[i].kbd = qkbd_state_init(con);
|
||||||
register_displaychangelistener(&sdl2_console[i].dcl);
|
register_displaychangelistener(&sdl2_console[i].dcl);
|
||||||
|
|
||||||
#if defined(SDL_VIDEO_DRIVER_WINDOWS) || defined(SDL_VIDEO_DRIVER_X11)
|
#if defined(SDL_VIDEO_DRIVER_WINDOWS) || defined(SDL_VIDEO_DRIVER_X11)
|
||||||
|
Loading…
Reference in New Issue
Block a user