diff options
author | Tony Crisci <tony@dubstepdish.com> | 2017-10-29 10:36:16 -0400 |
---|---|---|
committer | Tony Crisci <tony@dubstepdish.com> | 2017-10-29 10:36:16 -0400 |
commit | 88316e7921942e186df0c6f41fb405fe85bb83f9 (patch) | |
tree | 6e2b6a66fdaa578243b9a2584ab436849d1039ee | |
parent | 942a19a0695d1661e60d193b26661dd4aec70454 (diff) |
xwm: get rid of xcb call macro
-rw-r--r-- | xwayland/xwm.c | 38 |
1 files changed, 14 insertions, 24 deletions
diff --git a/xwayland/xwm.c b/xwayland/xwm.c index 3786b823..12dd5dd0 100644 --- a/xwayland/xwm.c +++ b/xwayland/xwm.c @@ -138,10 +138,10 @@ static void xwm_send_focus_window(struct wlr_xwm *xwm, uint32_t values[1]; values[0] = XCB_STACK_MODE_ABOVE; - xcb_configure_window_checked(xwm->xcb_conn, xsurface->window_id, + xcb_configure_window(xwm->xcb_conn, xsurface->window_id, XCB_CONFIG_WINDOW_STACK_MODE, values); } else { - xcb_set_input_focus_checked(xwm->xcb_conn, + xcb_set_input_focus(xwm->xcb_conn, XCB_INPUT_FOCUS_POINTER_ROOT, XCB_NONE, XCB_CURRENT_TIME); } @@ -223,21 +223,6 @@ static void wlr_xwayland_surface_destroy( free(xsurface); } -/* xcb helpers */ -#define XCB_CALL(xwm, x) xcb_call(xwm, __func__, __LINE__, x) -static bool xcb_call(struct wlr_xwm *xwm, const char *func, uint32_t line, - xcb_void_cookie_t cookie) { - xcb_generic_error_t *error; - if (!(error = xcb_request_check(xwm->xcb_conn, cookie))) { - return true; - } - - wlr_log(L_ERROR, "xcb call failed in %s:%u, x11 error code %d", - func, line, error->error_code); - free(error); - return false; -} - static void read_surface_class(struct wlr_xwm *xwm, struct wlr_xwayland_surface *surface, xcb_get_property_reply_t *reply) { if (reply->type != XCB_ATOM_STRING && @@ -1036,11 +1021,12 @@ void wlr_xwayland_surface_close(struct wlr_xwayland *wlr_xwayland, ev.type = xwm->atoms[WM_PROTOCOLS]; ev.data.data32[0] = xwm->atoms[WM_DELETE_WINDOW]; ev.data.data32[1] = XCB_CURRENT_TIME; - XCB_CALL(xwm, xcb_send_event_checked(xwm->xcb_conn, 0, - xsurface->window_id, XCB_EVENT_MASK_NO_EVENT, (char *)&ev)); + xcb_send_event(xwm->xcb_conn, 0, + xsurface->window_id, + XCB_EVENT_MASK_NO_EVENT, + (char *)&ev); } else { - XCB_CALL(xwm, xcb_kill_client_checked(xwm->xcb_conn, - xsurface->window_id)); + xcb_kill_client(xwm->xcb_conn, xsurface->window_id); } } @@ -1189,8 +1175,11 @@ static void xwm_get_visual_and_colormap(struct wlr_xwm *xwm) { xwm->visual_id = visualtype->visual_id; xwm->colormap = xcb_generate_id(xwm->xcb_conn); - xcb_create_colormap_checked(xwm->xcb_conn, XCB_COLORMAP_ALLOC_NONE, - xwm->colormap, xwm->screen->root, xwm->visual_id); + xcb_create_colormap(xwm->xcb_conn, + XCB_COLORMAP_ALLOC_NONE, + xwm->colormap, + xwm->screen->root, + xwm->visual_id); } struct wlr_xwm *xwm_create(struct wlr_xwayland *wlr_xwayland) { @@ -1241,7 +1230,8 @@ struct wlr_xwm *xwm_create(struct wlr_xwayland *wlr_xwayland) { XCB_CW_EVENT_MASK /* | XCB_CW_CURSOR */, values); - xcb_composite_redirect_subwindows_checked(xwm->xcb_conn, xwm->screen->root, + xcb_composite_redirect_subwindows(xwm->xcb_conn, + xwm->screen->root, XCB_COMPOSITE_REDIRECT_MANUAL); xcb_atom_t supported[] = { |