From 51a283cbe4a77d11853f7779eb1aa452557846d3 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Sun, 11 Nov 2018 11:48:45 +1300 Subject: backend/x11: Rename xcb_conn to xcb When the Xlib connection is removed, this _conn suffix is going to be pointless. I'm removing this preemtively for that. --- backend/x11/backend.c | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) (limited to 'backend/x11/backend.c') diff --git a/backend/x11/backend.c b/backend/x11/backend.c index cd7986a0..a9914879 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -81,7 +81,7 @@ static int x11_event(int fd, uint32_t mask, void *data) { } xcb_generic_event_t *e; - while ((e = xcb_poll_for_event(x11->xcb_conn))) { + while ((e = xcb_poll_for_event(x11->xcb))) { handle_x11_event(x11, e); free(e); } @@ -123,13 +123,13 @@ static bool backend_start(struct wlr_backend *backend) { }; for (size_t i = 0; i < sizeof(atom) / sizeof(atom[0]); ++i) { - atom[i].cookie = xcb_intern_atom(x11->xcb_conn, + atom[i].cookie = xcb_intern_atom(x11->xcb, true, strlen(atom[i].name), atom[i].name); } for (size_t i = 0; i < sizeof(atom) / sizeof(atom[0]); ++i) { xcb_intern_atom_reply_t *reply = xcb_intern_atom_reply( - x11->xcb_conn, atom[i].cookie, NULL); + x11->xcb, atom[i].cookie, NULL); if (reply) { *atom[i].atom = reply->atom; @@ -140,29 +140,29 @@ static bool backend_start(struct wlr_backend *backend) { } // create a blank cursor - xcb_pixmap_t pix = xcb_generate_id(x11->xcb_conn); - xcb_create_pixmap(x11->xcb_conn, 1, pix, x11->screen->root, 1, 1); + xcb_pixmap_t pix = xcb_generate_id(x11->xcb); + xcb_create_pixmap(x11->xcb, 1, pix, x11->screen->root, 1, 1); - x11->cursor = xcb_generate_id(x11->xcb_conn); - xcb_create_cursor(x11->xcb_conn, x11->cursor, pix, pix, 0, 0, 0, 0, 0, 0, + x11->cursor = xcb_generate_id(x11->xcb); + xcb_create_cursor(x11->xcb, x11->cursor, pix, pix, 0, 0, 0, 0, 0, 0, 0, 0); - xcb_free_pixmap(x11->xcb_conn, pix); + xcb_free_pixmap(x11->xcb, pix); #if WLR_HAS_XCB_XKB const xcb_query_extension_reply_t *reply = - xcb_get_extension_data(x11->xcb_conn, &xcb_xkb_id); + xcb_get_extension_data(x11->xcb, &xcb_xkb_id); if (reply != NULL && reply->present) { x11->xkb_base_event = reply->first_event; x11->xkb_base_error = reply->first_error; xcb_xkb_use_extension_cookie_t cookie = xcb_xkb_use_extension( - x11->xcb_conn, XCB_XKB_MAJOR_VERSION, XCB_XKB_MINOR_VERSION); + x11->xcb, XCB_XKB_MAJOR_VERSION, XCB_XKB_MINOR_VERSION); xcb_xkb_use_extension_reply_t *reply = - xcb_xkb_use_extension_reply(x11->xcb_conn, cookie, NULL); + xcb_xkb_use_extension_reply(x11->xcb, cookie, NULL); if (reply != NULL && reply->supported) { x11->xkb_supported = true; - xcb_xkb_select_events(x11->xcb_conn, + xcb_xkb_select_events(x11->xcb, XCB_XKB_ID_USE_CORE_KBD, XCB_XKB_EVENT_TYPE_STATE_NOTIFY, 0, @@ -210,7 +210,7 @@ static void backend_destroy(struct wlr_backend *backend) { wlr_egl_finish(&x11->egl); if (x11->cursor) { - xcb_free_cursor(x11->xcb_conn, x11->cursor); + xcb_free_cursor(x11->xcb, x11->cursor); } if (x11->xlib_conn) { XCloseDisplay(x11->xlib_conn); @@ -258,15 +258,15 @@ struct wlr_backend *wlr_x11_backend_create(struct wl_display *display, goto error_x11; } - x11->xcb_conn = XGetXCBConnection(x11->xlib_conn); - if (!x11->xcb_conn || xcb_connection_has_error(x11->xcb_conn)) { + x11->xcb = XGetXCBConnection(x11->xlib_conn); + if (!x11->xcb || xcb_connection_has_error(x11->xcb)) { wlr_log(WLR_ERROR, "Failed to open xcb connection"); goto error_display; } XSetEventQueueOwner(x11->xlib_conn, XCBOwnsEventQueue); - int fd = xcb_get_file_descriptor(x11->xcb_conn); + int fd = xcb_get_file_descriptor(x11->xcb); struct wl_event_loop *ev = wl_display_get_event_loop(display); uint32_t events = WL_EVENT_READABLE | WL_EVENT_ERROR | WL_EVENT_HANGUP; x11->event_source = wl_event_loop_add_fd(ev, fd, events, x11_event, x11); @@ -276,7 +276,7 @@ struct wlr_backend *wlr_x11_backend_create(struct wl_display *display, } wl_event_source_check(x11->event_source); - x11->screen = xcb_setup_roots_iterator(xcb_get_setup(x11->xcb_conn)).data; + x11->screen = xcb_setup_roots_iterator(xcb_get_setup(x11->xcb)).data; if (!create_renderer_func) { create_renderer_func = wlr_renderer_autocreate; -- cgit v1.2.3 From ab2c57984bd66b19dde6bf4c24f62faac9e8bd4d Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Sun, 11 Nov 2018 11:58:14 +1300 Subject: backend/x11: Make header order consistent --- backend/x11/backend.c | 19 ++++++++++++------- backend/x11/input_device.c | 15 ++++++++++----- backend/x11/output.c | 3 +++ include/backend/x11.h | 9 ++++++--- include/wlr/backend/x11.h | 2 ++ 5 files changed, 33 insertions(+), 15 deletions(-) (limited to 'backend/x11/backend.c') diff --git a/backend/x11/backend.c b/backend/x11/backend.c index a9914879..2422b8e8 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -1,25 +1,30 @@ #define _POSIX_C_SOURCE 200112L + #include #include #include #include #include #include + +#include + +#include #include +#include +#if WLR_HAS_XCB_XKB +#include +#endif + #include #include -#include #include #include #include #include -#include +#include #include -#include -#include -#if WLR_HAS_XCB_XKB -#include -#endif + #include "backend/x11.h" #include "util/signal.h" diff --git a/backend/x11/input_device.c b/backend/x11/input_device.c index b64151e0..fa19221f 100644 --- a/backend/x11/input_device.c +++ b/backend/x11/input_device.c @@ -1,18 +1,23 @@ #include + #include -#include -#include -#include -#include -#include + #ifdef __linux__ #include #elif __FreeBSD__ #include #endif + +#include #if WLR_HAS_XCB_XKB #include #endif + +#include +#include +#include +#include + #include "backend/x11.h" #include "util/signal.h" diff --git a/backend/x11/output.c b/backend/x11/output.c index 92eeca66..8cb4b3eb 100644 --- a/backend/x11/output.c +++ b/backend/x11/output.c @@ -1,10 +1,13 @@ #define _POSIX_C_SOURCE 200809L + #include #include #include + #include #include #include + #include "backend/x11.h" #include "util/signal.h" diff --git a/include/backend/x11.h b/include/backend/x11.h index cc7850b1..eccb422b 100644 --- a/include/backend/x11.h +++ b/include/backend/x11.h @@ -2,14 +2,17 @@ #define BACKEND_X11_H #include + +#include #include -#include +#include + #include +#include #include #include #include -#include -#include +#include #define XCB_EVENT_RESPONSE_TYPE_MASK 0x7f diff --git a/include/wlr/backend/x11.h b/include/wlr/backend/x11.h index 56360bf7..5793a3b9 100644 --- a/include/wlr/backend/x11.h +++ b/include/wlr/backend/x11.h @@ -2,7 +2,9 @@ #define WLR_BACKEND_X11_H #include + #include + #include #include #include -- cgit v1.2.3 From 9c1b87f210edd17a3006ff9fe90be70e1c31a446 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Sun, 11 Nov 2018 12:12:50 +1300 Subject: backend/x11: Move atom initilisation earlier There isn't any real reason to delay this until the backend is started. --- backend/x11/backend.c | 68 +++++++++++++++++++++------------------------------ 1 file changed, 28 insertions(+), 40 deletions(-) (limited to 'backend/x11/backend.c') diff --git a/backend/x11/backend.c b/backend/x11/backend.c index 2422b8e8..89e2b867 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -104,46 +104,6 @@ static bool backend_start(struct wlr_backend *backend) { struct wlr_x11_backend *x11 = get_x11_backend_from_backend(backend); x11->started = true; - struct { - const char *name; - xcb_intern_atom_cookie_t cookie; - xcb_atom_t *atom; - } atom[] = { - { - .name = "WM_PROTOCOLS", - .atom = &x11->atoms.wm_protocols, - }, - { - .name = "WM_DELETE_WINDOW", - .atom = &x11->atoms.wm_delete_window, - }, - { - .name = "_NET_WM_NAME", - .atom = &x11->atoms.net_wm_name, - }, - { - .name = "UTF8_STRING", - .atom = &x11->atoms.utf8_string, - }, - }; - - for (size_t i = 0; i < sizeof(atom) / sizeof(atom[0]); ++i) { - atom[i].cookie = xcb_intern_atom(x11->xcb, - true, strlen(atom[i].name), atom[i].name); - } - - for (size_t i = 0; i < sizeof(atom) / sizeof(atom[0]); ++i) { - xcb_intern_atom_reply_t *reply = xcb_intern_atom_reply( - x11->xcb, atom[i].cookie, NULL); - - if (reply) { - *atom[i].atom = reply->atom; - free(reply); - } else { - *atom[i].atom = XCB_ATOM_NONE; - } - } - // create a blank cursor xcb_pixmap_t pix = xcb_generate_id(x11->xcb); xcb_create_pixmap(x11->xcb, 1, pix, x11->screen->root, 1, 1); @@ -271,6 +231,34 @@ struct wlr_backend *wlr_x11_backend_create(struct wl_display *display, XSetEventQueueOwner(x11->xlib_conn, XCBOwnsEventQueue); + struct { + const char *name; + xcb_intern_atom_cookie_t cookie; + xcb_atom_t *atom; + } atom[] = { + { .name = "WM_PROTOCOLS", .atom = &x11->atoms.wm_protocols }, + { .name = "WM_DELETE_WINDOW", .atom = &x11->atoms.wm_delete_window }, + { .name = "_NET_WM_NAME", .atom = &x11->atoms.net_wm_name }, + { .name = "UTF8_STRING", .atom = &x11->atoms.utf8_string }, + }; + + for (size_t i = 0; i < sizeof(atom) / sizeof(atom[0]); ++i) { + atom[i].cookie = xcb_intern_atom(x11->xcb, + true, strlen(atom[i].name), atom[i].name); + } + + for (size_t i = 0; i < sizeof(atom) / sizeof(atom[0]); ++i) { + xcb_intern_atom_reply_t *reply = xcb_intern_atom_reply( + x11->xcb, atom[i].cookie, NULL); + + if (reply) { + *atom[i].atom = reply->atom; + free(reply); + } else { + *atom[i].atom = XCB_ATOM_NONE; + } + } + int fd = xcb_get_file_descriptor(x11->xcb); struct wl_event_loop *ev = wl_display_get_event_loop(display); uint32_t events = WL_EVENT_READABLE | WL_EVENT_ERROR | WL_EVENT_HANGUP; -- cgit v1.2.3 From 88b2d6fe25f77b2140bb76d5d3c8ae037cad31e5 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Sun, 11 Nov 2018 12:30:37 +1300 Subject: backend/x11: Check for xinput extension --- README.md | 1 + backend/x11/backend.c | 27 +++++++++++++++++++++++++++ backend/x11/meson.build | 3 ++- include/backend/x11.h | 2 ++ 4 files changed, 32 insertions(+), 1 deletion(-) (limited to 'backend/x11/backend.c') diff --git a/README.md b/README.md index 1519d0b3..9357cc59 100644 --- a/README.md +++ b/README.md @@ -62,6 +62,7 @@ If you choose to enable X11 support: * xcb * xcb-composite * xcb-xfixes +* xcb-xinput * xcb-image * xcb-render * x11-xcb diff --git a/backend/x11/backend.c b/backend/x11/backend.c index 89e2b867..d4e228b1 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -12,6 +12,7 @@ #include #include #include +#include #if WLR_HAS_XCB_XKB #include #endif @@ -74,6 +75,12 @@ static void handle_x11_event(struct wlr_x11_backend *x11, } break; } + case XCB_GE_GENERIC: { + xcb_ge_generic_event_t *ev = (xcb_ge_generic_event_t *)event; + if (ev->extension == x11->xinput_opcode) { + handle_x11_xinput_event(x11, ev); + } + } } } @@ -259,6 +266,26 @@ struct wlr_backend *wlr_x11_backend_create(struct wl_display *display, } } + const xcb_query_extension_reply_t *ext = + xcb_get_extension_data(x11->xcb, &xcb_input_id); + if (!ext || !ext->present) { + wlr_log(WLR_ERROR, "X11 does not support Xinput extension"); + goto error_display; + } + x11->xinput_opcode = ext->major_opcode; + + xcb_input_xi_query_version_cookie_t xi_cookie = + xcb_input_xi_query_version(x11->xcb, 2, 0); + xcb_input_xi_query_version_reply_t *xi_reply = + xcb_input_xi_query_version_reply(x11->xcb, xi_cookie, NULL); + + if (!xi_reply || xi_reply->major_version < 2) { + wlr_log(WLR_ERROR, "X11 does not support required Xinput version"); + free(xi_reply); + goto error_display; + } + free(xi_reply); + int fd = xcb_get_file_descriptor(x11->xcb); struct wl_event_loop *ev = wl_display_get_event_loop(display); uint32_t events = WL_EVENT_READABLE | WL_EVENT_ERROR | WL_EVENT_HANGUP; diff --git a/backend/x11/meson.build b/backend/x11/meson.build index fa51e3d7..04111a13 100644 --- a/backend/x11/meson.build +++ b/backend/x11/meson.build @@ -1,7 +1,8 @@ x11_libs = [] x11_required = [ - 'xcb', 'x11-xcb', + 'xcb', + 'xcb-xinput', ] x11_optional = [ 'xcb-xkb', diff --git a/include/backend/x11.h b/include/backend/x11.h index eccb422b..935c427e 100644 --- a/include/backend/x11.h +++ b/include/backend/x11.h @@ -67,6 +67,8 @@ struct wlr_x11_backend { // A blank cursor xcb_cursor_t cursor; + uint8_t xinput_opcode; + #if WLR_HAS_XCB_XKB bool xkb_supported; uint8_t xkb_base_event; -- cgit v1.2.3 From 70ae76304e5c40774dd50a23537cdc419cdfa4c6 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Sun, 11 Nov 2018 14:25:53 +1300 Subject: backend/x11: Remove old input functions This also allows us to remove xcb-xkb, since Xinput will now give us the appropriate XKB modifiers with each event. --- README.md | 1 - backend/x11/backend.c | 33 ----------- backend/x11/input_device.c | 138 +++++---------------------------------------- backend/x11/meson.build | 11 ---- backend/x11/output.c | 8 +-- include/backend/x11.h | 8 --- include/wlr/config.h.in | 1 - meson.build | 1 - meson_options.txt | 1 - 9 files changed, 14 insertions(+), 188 deletions(-) (limited to 'backend/x11/backend.c') diff --git a/README.md b/README.md index 9357cc59..a2fca488 100644 --- a/README.md +++ b/README.md @@ -68,7 +68,6 @@ If you choose to enable X11 support: * x11-xcb * xcb-errors (optional, for improved error reporting) * x11-icccm (optional, for improved Xwayland introspection) -* xcb-xkb (optional, for improved keyboard handling on the X11 backend) Run these commands: diff --git a/backend/x11/backend.c b/backend/x11/backend.c index d4e228b1..ce747976 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -13,9 +13,6 @@ #include #include #include -#if WLR_HAS_XCB_XKB -#include -#endif #include #include @@ -42,8 +39,6 @@ struct wlr_x11_output *get_x11_output_from_window_id( static void handle_x11_event(struct wlr_x11_backend *x11, xcb_generic_event_t *event) { - handle_x11_input_event(x11, event); - switch (event->response_type & XCB_EVENT_RESPONSE_TYPE_MASK) { case XCB_EXPOSE: { xcb_expose_event_t *ev = (xcb_expose_event_t *)event; @@ -120,34 +115,6 @@ static bool backend_start(struct wlr_backend *backend) { 0, 0); xcb_free_pixmap(x11->xcb, pix); -#if WLR_HAS_XCB_XKB - const xcb_query_extension_reply_t *reply = - xcb_get_extension_data(x11->xcb, &xcb_xkb_id); - if (reply != NULL && reply->present) { - x11->xkb_base_event = reply->first_event; - x11->xkb_base_error = reply->first_error; - - xcb_xkb_use_extension_cookie_t cookie = xcb_xkb_use_extension( - x11->xcb, XCB_XKB_MAJOR_VERSION, XCB_XKB_MINOR_VERSION); - xcb_xkb_use_extension_reply_t *reply = - xcb_xkb_use_extension_reply(x11->xcb, cookie, NULL); - if (reply != NULL && reply->supported) { - x11->xkb_supported = true; - - xcb_xkb_select_events(x11->xcb, - XCB_XKB_ID_USE_CORE_KBD, - XCB_XKB_EVENT_TYPE_STATE_NOTIFY, - 0, - XCB_XKB_EVENT_TYPE_STATE_NOTIFY, - 0, - 0, - 0); - - free(reply); - } - } -#endif - wlr_signal_emit_safe(&x11->backend.events.new_input, &x11->keyboard_dev); for (size_t i = 0; i < x11->requested_outputs; ++i) { diff --git a/backend/x11/input_device.c b/backend/x11/input_device.c index 6a0de1e8..08d982e6 100644 --- a/backend/x11/input_device.c +++ b/backend/x11/input_device.c @@ -10,9 +10,6 @@ #include #include -#if WLR_HAS_XCB_XKB -#include -#endif #include #include @@ -22,126 +19,6 @@ #include "backend/x11.h" #include "util/signal.h" -static uint32_t xcb_button_to_wl(uint32_t button) { - switch (button) { - case XCB_BUTTON_INDEX_1: return BTN_LEFT; - case XCB_BUTTON_INDEX_2: return BTN_MIDDLE; - case XCB_BUTTON_INDEX_3: return BTN_RIGHT; - case XCB_BUTTON_INDEX_4: return BTN_GEAR_UP; - case XCB_BUTTON_INDEX_5: return BTN_GEAR_DOWN; - default: return 0; - } -} - -static void x11_handle_pointer_position(struct wlr_x11_output *output, - int16_t x, int16_t y, xcb_timestamp_t time) { - struct wlr_x11_backend *x11 = output->x11; - struct wlr_output *wlr_output = &output->wlr_output; - struct wlr_event_pointer_motion_absolute event = { - .device = &output->pointer_dev, - .time_msec = time, - .x = (double)x / wlr_output->width, - .y = (double)y / wlr_output->height, - }; - wlr_signal_emit_safe(&output->pointer.events.motion_absolute, &event); - - x11->time = time; -} - -void handle_x11_input_event(struct wlr_x11_backend *x11, - xcb_generic_event_t *event) { - switch (event->response_type & XCB_EVENT_RESPONSE_TYPE_MASK) { - case XCB_KEY_PRESS: - case XCB_KEY_RELEASE: { - xcb_key_press_event_t *ev = (xcb_key_press_event_t *)event; - struct wlr_event_keyboard_key key = { - .time_msec = ev->time, - .keycode = ev->detail - 8, - .state = event->response_type == XCB_KEY_PRESS ? - WLR_KEY_PRESSED : WLR_KEY_RELEASED, - .update_state = true, - }; - - // TODO use xcb-xkb for more precise modifiers state? - wlr_keyboard_notify_key(&x11->keyboard, &key); - x11->time = ev->time; - return; - } - case XCB_BUTTON_PRESS: { - xcb_button_press_event_t *ev = (xcb_button_press_event_t *)event; - - struct wlr_x11_output *output = - get_x11_output_from_window_id(x11, ev->event); - if (output == NULL) { - break; - } - - if (ev->detail == XCB_BUTTON_INDEX_4 || - ev->detail == XCB_BUTTON_INDEX_5) { - int32_t delta_discrete = ev->detail == XCB_BUTTON_INDEX_4 ? -1 : 1; - struct wlr_event_pointer_axis axis = { - .device = &output->pointer_dev, - .time_msec = ev->time, - .source = WLR_AXIS_SOURCE_WHEEL, - .orientation = WLR_AXIS_ORIENTATION_VERTICAL, - // 15 is a typical value libinput sends for one scroll - .delta = delta_discrete * 15, - .delta_discrete = delta_discrete, - }; - wlr_signal_emit_safe(&output->pointer.events.axis, &axis); - x11->time = ev->time; - break; - } - } - /* fallthrough */ - case XCB_BUTTON_RELEASE: { - xcb_button_press_event_t *ev = (xcb_button_press_event_t *)event; - - struct wlr_x11_output *output = - get_x11_output_from_window_id(x11, ev->event); - if (output == NULL) { - break; - } - - if (ev->detail != XCB_BUTTON_INDEX_4 && - ev->detail != XCB_BUTTON_INDEX_5) { - struct wlr_event_pointer_button button = { - .device = &output->pointer_dev, - .time_msec = ev->time, - .button = xcb_button_to_wl(ev->detail), - .state = event->response_type == XCB_BUTTON_PRESS ? - WLR_BUTTON_PRESSED : WLR_BUTTON_RELEASED, - }; - - wlr_signal_emit_safe(&output->pointer.events.button, &button); - } - x11->time = ev->time; - return; - } - case XCB_MOTION_NOTIFY: { - xcb_motion_notify_event_t *ev = (xcb_motion_notify_event_t *)event; - - struct wlr_x11_output *output = - get_x11_output_from_window_id(x11, ev->event); - if (output != NULL) { - x11_handle_pointer_position(output, ev->event_x, ev->event_y, ev->time); - } - return; - } - default: -#if WLR_HAS_XCB_XKB - if (x11->xkb_supported && event->response_type == x11->xkb_base_event) { - xcb_xkb_state_notify_event_t *ev = - (xcb_xkb_state_notify_event_t *)event; - wlr_keyboard_notify_modifiers(&x11->keyboard, ev->baseMods, - ev->latchedMods, ev->lockedMods, ev->lockedGroup); - return; - } -#endif - break; - } -} - static void send_key_event(struct wlr_x11_backend *x11, uint32_t key, enum wlr_key_state st, xcb_timestamp_t time) { struct wlr_event_keyboard_key ev = { @@ -178,6 +55,17 @@ static void send_axis_event(struct wlr_x11_output *output, int32_t delta, wlr_signal_emit_safe(&output->pointer.events.axis, &ev); } +static void send_pointer_position_event(struct wlr_x11_output *output, + int16_t x, int16_t y, xcb_timestamp_t time) { + struct wlr_event_pointer_motion_absolute ev = { + .device = &output->pointer_dev, + .time_msec = time, + .x = (double)x / output->wlr_output.width, + .y = (double)y / output->wlr_output.height, + }; + wlr_signal_emit_safe(&output->pointer.events.motion_absolute, &ev); +} + void handle_x11_xinput_event(struct wlr_x11_backend *x11, xcb_ge_generic_event_t *event) { struct wlr_x11_output *output; @@ -271,7 +159,7 @@ void handle_x11_xinput_event(struct wlr_x11_backend *x11, return; } - x11_handle_pointer_position(output, ev->event_x >> 16, + send_pointer_position_event(output, ev->event_x >> 16, ev->event_y >> 16, ev->time); x11->time = ev->time; break; @@ -315,7 +203,7 @@ void update_x11_pointer_position(struct wlr_x11_output *output, return; } - x11_handle_pointer_position(output, reply->win_x, reply->win_y, time); + send_pointer_position_event(output, reply->win_x, reply->win_y, time); free(reply); } diff --git a/backend/x11/meson.build b/backend/x11/meson.build index 04111a13..d9fd8d91 100644 --- a/backend/x11/meson.build +++ b/backend/x11/meson.build @@ -4,9 +4,6 @@ x11_required = [ 'xcb', 'xcb-xinput', ] -x11_optional = [ - 'xcb-xkb', -] foreach lib : x11_required dep = dependency(lib, required: get_option('x11-backend')) @@ -17,14 +14,6 @@ foreach lib : x11_required x11_libs += dep endforeach -foreach lib : x11_optional - dep = dependency(lib, required: get_option(lib)) - if dep.found() - x11_libs += dep - conf_data.set10('WLR_HAS_' + lib.underscorify().to_upper(), true) - endif -endforeach - lib_wlr_backend_x11 = static_library( 'wlr_backend_x11', files( diff --git a/backend/x11/output.c b/backend/x11/output.c index 3acb41fd..ba70d5cb 100644 --- a/backend/x11/output.c +++ b/backend/x11/output.c @@ -152,18 +152,13 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { uint32_t mask = XCB_CW_EVENT_MASK; uint32_t values[] = { - XCB_EVENT_MASK_EXPOSURE | - XCB_EVENT_MASK_KEY_PRESS | XCB_EVENT_MASK_KEY_RELEASE | - XCB_EVENT_MASK_BUTTON_PRESS | XCB_EVENT_MASK_BUTTON_RELEASE | - XCB_EVENT_MASK_POINTER_MOTION | - XCB_EVENT_MASK_STRUCTURE_NOTIFY + XCB_EVENT_MASK_EXPOSURE | XCB_EVENT_MASK_STRUCTURE_NOTIFY }; output->win = xcb_generate_id(x11->xcb); xcb_create_window(x11->xcb, XCB_COPY_FROM_PARENT, output->win, x11->screen->root, 0, 0, wlr_output->width, wlr_output->height, 1, XCB_WINDOW_CLASS_INPUT_OUTPUT, x11->screen->root_visual, mask, values); -#if 0 struct { xcb_input_event_mask_t head; xcb_input_xi_event_mask_t mask; @@ -176,7 +171,6 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { XCB_INPUT_XI_EVENT_MASK_MOTION, }; xcb_input_xi_select_events(x11->xcb, output->win, 1, &xinput_mask.head); -#endif output->surf = wlr_egl_create_surface(&x11->egl, &output->win); if (!output->surf) { diff --git a/include/backend/x11.h b/include/backend/x11.h index ac0930b6..94de129f 100644 --- a/include/backend/x11.h +++ b/include/backend/x11.h @@ -69,12 +69,6 @@ struct wlr_x11_backend { uint8_t xinput_opcode; -#if WLR_HAS_XCB_XKB - bool xkb_supported; - uint8_t xkb_base_event; - uint8_t xkb_base_error; -#endif - struct wl_listener display_destroy; }; @@ -87,8 +81,6 @@ extern const struct wlr_keyboard_impl keyboard_impl; extern const struct wlr_pointer_impl pointer_impl; extern const struct wlr_input_device_impl input_device_impl; -void handle_x11_input_event(struct wlr_x11_backend *x11, - xcb_generic_event_t *event); void handle_x11_xinput_event(struct wlr_x11_backend *x11, xcb_ge_generic_event_t *event); void update_x11_pointer_position(struct wlr_x11_output *output, diff --git a/include/wlr/config.h.in b/include/wlr/config.h.in index 17277c07..94273fac 100644 --- a/include/wlr/config.h.in +++ b/include/wlr/config.h.in @@ -12,6 +12,5 @@ #mesondefine WLR_HAS_XCB_ERRORS #mesondefine WLR_HAS_XCB_ICCCM -#mesondefine WLR_HAS_XCB_XKB #endif diff --git a/meson.build b/meson.build index 7017e4b5..3d96d052 100644 --- a/meson.build +++ b/meson.build @@ -35,7 +35,6 @@ conf_data.set10('WLR_HAS_X11_BACKEND', false) conf_data.set10('WLR_HAS_XWAYLAND', false) conf_data.set10('WLR_HAS_XCB_ERRORS', false) conf_data.set10('WLR_HAS_XCB_ICCCM', false) -conf_data.set10('WLR_HAS_XCB_XKB', false) wlr_inc = include_directories('.', 'include') diff --git a/meson_options.txt b/meson_options.txt index 360c6f6a..19a6cad7 100644 --- a/meson_options.txt +++ b/meson_options.txt @@ -3,7 +3,6 @@ option('logind', type: 'feature', value: 'auto', description: 'Enable support fo option('logind-provider', type: 'combo', choices: ['systemd', 'elogind'], value: 'systemd', description: 'Provider of logind support library') option('xcb-errors', type: 'feature', value: 'auto', description: 'Use xcb-errors util library') option('xcb-icccm', type: 'feature', value: 'auto', description: 'Use xcb-icccm util library') -option('xcb-xkb', type: 'feature', value: 'auto', description: 'Use xcb-xkb util library') option('xwayland', type: 'feature', value: 'auto', description: 'Enable support for X11 applications') option('x11-backend', type: 'feature', value: 'auto', description: 'Enable X11 backend') option('rootston', type: 'boolean', value: true, description: 'Build the rootston example compositor') -- cgit v1.2.3 From d3ee69f76bbc8b71a79840c047fa7a6144ddc628 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Sun, 11 Nov 2018 14:51:22 +1300 Subject: backend/x11: Hide cursor with Xfixes --- backend/x11/backend.c | 36 ++++++++++++++++++++++-------------- backend/x11/input_device.c | 31 +++++++++++++++++++++++++++++++ backend/x11/meson.build | 1 + backend/x11/output.c | 8 +++----- include/backend/x11.h | 5 ++--- 5 files changed, 59 insertions(+), 22 deletions(-) (limited to 'backend/x11/backend.c') diff --git a/backend/x11/backend.c b/backend/x11/backend.c index ce747976..38715631 100644 --- a/backend/x11/backend.c +++ b/backend/x11/backend.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include @@ -106,15 +107,6 @@ static bool backend_start(struct wlr_backend *backend) { struct wlr_x11_backend *x11 = get_x11_backend_from_backend(backend); x11->started = true; - // create a blank cursor - xcb_pixmap_t pix = xcb_generate_id(x11->xcb); - xcb_create_pixmap(x11->xcb, 1, pix, x11->screen->root, 1, 1); - - x11->cursor = xcb_generate_id(x11->xcb); - xcb_create_cursor(x11->xcb, x11->cursor, pix, pix, 0, 0, 0, 0, 0, 0, - 0, 0); - xcb_free_pixmap(x11->xcb, pix); - wlr_signal_emit_safe(&x11->backend.events.new_input, &x11->keyboard_dev); for (size_t i = 0; i < x11->requested_outputs; ++i) { @@ -148,9 +140,6 @@ static void backend_destroy(struct wlr_backend *backend) { wlr_renderer_destroy(x11->renderer); wlr_egl_finish(&x11->egl); - if (x11->cursor) { - xcb_free_cursor(x11->xcb, x11->cursor); - } if (x11->xlib_conn) { XCloseDisplay(x11->xlib_conn); } @@ -233,8 +222,27 @@ struct wlr_backend *wlr_x11_backend_create(struct wl_display *display, } } - const xcb_query_extension_reply_t *ext = - xcb_get_extension_data(x11->xcb, &xcb_input_id); + const xcb_query_extension_reply_t *ext; + + ext = xcb_get_extension_data(x11->xcb, &xcb_xfixes_id); + if (!ext || !ext->present) { + wlr_log(WLR_ERROR, "X11 does not support Xfixes extension"); + goto error_display; + } + + xcb_xfixes_query_version_cookie_t fixes_cookie = + xcb_xfixes_query_version(x11->xcb, 4, 0); + xcb_xfixes_query_version_reply_t *fixes_reply = + xcb_xfixes_query_version_reply(x11->xcb, fixes_cookie, NULL); + + if (!fixes_reply || fixes_reply->major_version < 4) { + wlr_log(WLR_ERROR, "X11 does not support required Xfixes version"); + free(fixes_reply); + goto error_display; + } + free(fixes_reply); + + ext = xcb_get_extension_data(x11->xcb, &xcb_input_id); if (!ext || !ext->present) { wlr_log(WLR_ERROR, "X11 does not support Xinput extension"); goto error_display; diff --git a/backend/x11/input_device.c b/backend/x11/input_device.c index 08d982e6..a50f478a 100644 --- a/backend/x11/input_device.c +++ b/backend/x11/input_device.c @@ -9,6 +9,7 @@ #endif #include +#include #include #include @@ -164,6 +165,36 @@ void handle_x11_xinput_event(struct wlr_x11_backend *x11, x11->time = ev->time; break; } + case XCB_INPUT_ENTER: { + xcb_input_enter_event_t *ev = (xcb_input_enter_event_t *)event; + + output = get_x11_output_from_window_id(x11, ev->event); + if (!output) { + return; + } + + if (!output->cursor_hidden) { + xcb_xfixes_hide_cursor(x11->xcb, output->win); + xcb_flush(x11->xcb); + output->cursor_hidden = true; + } + break; + } + case XCB_INPUT_LEAVE: { + xcb_input_leave_event_t *ev = (xcb_input_leave_event_t *)event; + + output = get_x11_output_from_window_id(x11, ev->event); + if (!output) { + return; + } + + if (output->cursor_hidden) { + xcb_xfixes_show_cursor(x11->xcb, output->win); + xcb_flush(x11->xcb); + output->cursor_hidden = false; + } + break; + } } } diff --git a/backend/x11/meson.build b/backend/x11/meson.build index d9fd8d91..19e873ab 100644 --- a/backend/x11/meson.build +++ b/backend/x11/meson.build @@ -3,6 +3,7 @@ x11_required = [ 'x11-xcb', 'xcb', 'xcb-xinput', + 'xcb-xfixes', ] foreach lib : x11_required diff --git a/backend/x11/output.c b/backend/x11/output.c index ba70d5cb..6f98c590 100644 --- a/backend/x11/output.c +++ b/backend/x11/output.c @@ -168,7 +168,9 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { XCB_INPUT_XI_EVENT_MASK_KEY_RELEASE | XCB_INPUT_XI_EVENT_MASK_BUTTON_PRESS | XCB_INPUT_XI_EVENT_MASK_BUTTON_RELEASE | - XCB_INPUT_XI_EVENT_MASK_MOTION, + XCB_INPUT_XI_EVENT_MASK_MOTION | + XCB_INPUT_XI_EVENT_MASK_ENTER | + XCB_INPUT_XI_EVENT_MASK_LEAVE, }; xcb_input_xi_select_events(x11->xcb, output->win, 1, &xinput_mask.head); @@ -190,10 +192,6 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { strlen(title), title); } - uint32_t cursor_values[] = { x11->cursor }; - xcb_change_window_attributes(x11->xcb, output->win, XCB_CW_CURSOR, - cursor_values); - xcb_map_window(x11->xcb, output->win); xcb_flush(x11->xcb); diff --git a/include/backend/x11.h b/include/backend/x11.h index 94de129f..1a8341f6 100644 --- a/include/backend/x11.h +++ b/include/backend/x11.h @@ -33,6 +33,8 @@ struct wlr_x11_output { struct wl_event_source *frame_timer; int frame_delay; + + bool cursor_hidden; }; struct wlr_x11_backend { @@ -64,9 +66,6 @@ struct wlr_x11_backend { // The time we last received an event xcb_timestamp_t time; - // A blank cursor - xcb_cursor_t cursor; - uint8_t xinput_opcode; struct wl_listener display_destroy; -- cgit v1.2.3