Send separate DOWN/UP key events

Shortcuts generated instant DOWN/UP key events. Instead, generate DOWN
event on Ctrl+key down and UP event on Ctrl+key up.

Fixes <https://github.com/Genymobile/scrcpy/issues/166>.
This commit is contained in:
Romain Vimont 2018-06-24 21:09:23 +02:00
parent 2f66acd75d
commit c12c64ed41

View file

@ -29,52 +29,58 @@ static struct point get_mouse_point(struct screen *screen) {
}; };
} }
static void send_keycode(struct controller *controller, enum android_keycode keycode, const char *name) { static const int ACTION_DOWN = 1;
static const int ACTION_UP = 1 << 1;
static void send_keycode(struct controller *controller, enum android_keycode keycode, int actions, const char *name) {
// send DOWN event // send DOWN event
struct control_event control_event; struct control_event control_event;
control_event.type = CONTROL_EVENT_TYPE_KEYCODE; control_event.type = CONTROL_EVENT_TYPE_KEYCODE;
control_event.keycode_event.action = AKEY_EVENT_ACTION_DOWN;
control_event.keycode_event.keycode = keycode; control_event.keycode_event.keycode = keycode;
control_event.keycode_event.metastate = 0; control_event.keycode_event.metastate = 0;
if (!controller_push_event(controller, &control_event)) { if (actions & ACTION_DOWN) {
LOGW("Cannot send %s (DOWN)", name); control_event.keycode_event.action = AKEY_EVENT_ACTION_DOWN;
return; if (!controller_push_event(controller, &control_event)) {
LOGW("Cannot send %s (DOWN)", name);
return;
}
} }
// send UP event if (actions & ACTION_UP) {
control_event.keycode_event.action = AKEY_EVENT_ACTION_UP; control_event.keycode_event.action = AKEY_EVENT_ACTION_UP;
if (!controller_push_event(controller, &control_event)) { if (!controller_push_event(controller, &control_event)) {
LOGW("Cannot send %s (UP)", name); LOGW("Cannot send %s (UP)", name);
}
} }
} }
static inline void action_home(struct controller *controller) { static inline void action_home(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_HOME, "HOME"); send_keycode(controller, AKEYCODE_HOME, actions, "HOME");
} }
static inline void action_back(struct controller *controller) { static inline void action_back(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_BACK, "BACK"); send_keycode(controller, AKEYCODE_BACK, actions, "BACK");
} }
static inline void action_app_switch(struct controller *controller) { static inline void action_app_switch(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_APP_SWITCH, "APP_SWITCH"); send_keycode(controller, AKEYCODE_APP_SWITCH, actions, "APP_SWITCH");
} }
static inline void action_power(struct controller *controller) { static inline void action_power(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_POWER, "POWER"); send_keycode(controller, AKEYCODE_POWER, actions, "POWER");
} }
static inline void action_volume_up(struct controller *controller) { static inline void action_volume_up(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_VOLUME_UP, "VOLUME_UP"); send_keycode(controller, AKEYCODE_VOLUME_UP, actions, "VOLUME_UP");
} }
static inline void action_volume_down(struct controller *controller) { static inline void action_volume_down(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_VOLUME_DOWN, "VOLUME_DOWN"); send_keycode(controller, AKEYCODE_VOLUME_DOWN, actions, "VOLUME_DOWN");
} }
static inline void action_menu(struct controller *controller) { static inline void action_menu(struct controller *controller, int actions) {
send_keycode(controller, AKEYCODE_MENU, "MENU"); send_keycode(controller, AKEYCODE_MENU, actions, "MENU");
} }
// turn the screen on if it was off, press BACK otherwise // turn the screen on if it was off, press BACK otherwise
@ -143,8 +149,8 @@ void input_manager_process_key(struct input_manager *input_manager,
if (ctrl) { if (ctrl) {
SDL_bool repeat = event->repeat; SDL_bool repeat = event->repeat;
// only consider keydown events, and ignore repeated events // ignore repeated events
if (repeat || event->type != SDL_KEYDOWN) { if (repeat) {
return; return;
} }
@ -155,43 +161,54 @@ void input_manager_process_key(struct input_manager *input_manager,
} }
SDL_Keycode keycode = event->keysym.sym; SDL_Keycode keycode = event->keysym.sym;
int action = event->type == SDL_KEYDOWN ? ACTION_DOWN : ACTION_UP;
switch (keycode) { switch (keycode) {
case SDLK_h: case SDLK_h:
action_home(input_manager->controller); action_home(input_manager->controller, action);
return; return;
case SDLK_b: // fall-through case SDLK_b: // fall-through
case SDLK_BACKSPACE: case SDLK_BACKSPACE:
action_back(input_manager->controller); action_back(input_manager->controller, action);
return; return;
case SDLK_s: case SDLK_s:
action_app_switch(input_manager->controller); action_app_switch(input_manager->controller, action);
return; return;
case SDLK_m: case SDLK_m:
action_menu(input_manager->controller); action_menu(input_manager->controller, action);
return; return;
case SDLK_p: case SDLK_p:
action_power(input_manager->controller); action_power(input_manager->controller, action);
return; return;
case SDLK_DOWN: case SDLK_DOWN:
action_volume_down(input_manager->controller); action_volume_down(input_manager->controller, action);
return; return;
case SDLK_UP: case SDLK_UP:
action_volume_up(input_manager->controller); action_volume_up(input_manager->controller, action);
return; return;
case SDLK_v: case SDLK_v:
clipboard_paste(input_manager->controller); if (event->type == SDL_KEYDOWN) {
clipboard_paste(input_manager->controller);
}
return; return;
case SDLK_f: case SDLK_f:
screen_switch_fullscreen(input_manager->screen); if (event->type == SDL_KEYDOWN) {
screen_switch_fullscreen(input_manager->screen);
}
return; return;
case SDLK_x: case SDLK_x:
screen_resize_to_fit(input_manager->screen); if (event->type == SDL_KEYDOWN) {
screen_resize_to_fit(input_manager->screen);
}
return; return;
case SDLK_g: case SDLK_g:
screen_resize_to_pixel_perfect(input_manager->screen); if (event->type == SDL_KEYDOWN) {
screen_resize_to_pixel_perfect(input_manager->screen);
}
return; return;
case SDLK_i: case SDLK_i:
switch_fps_counter_state(input_manager->frames); if (event->type == SDL_KEYDOWN) {
switch_fps_counter_state(input_manager->frames);
}
return; return;
} }
@ -228,7 +245,7 @@ void input_manager_process_mouse_button(struct input_manager *input_manager,
return; return;
} }
if (event->button == SDL_BUTTON_MIDDLE) { if (event->button == SDL_BUTTON_MIDDLE) {
action_home(input_manager->controller); action_home(input_manager->controller, ACTION_DOWN | ACTION_UP);
return; return;
} }
// double-click on black borders resize to fit the device screen // double-click on black borders resize to fit the device screen