xcb: don’t use the _unchecked variant for requests which have a reply

diff --git a/src/manage.c b/src/manage.c
index 3636b1adb4578fe03fde45bd537e3d05c7ca4a99..06088a82f7956cbead343a0f3ae5a6573e264ed7 100644 (file)
--- a/src/manage.c
+++ b/src/manage.c
@@ -126,7 +126,7 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki
     values[0] = XCB_EVENT_MASK_PROPERTY_CHANGE;
     xcb_change_window_attributes(conn, window, XCB_CW_EVENT_MASK, values);
 
-#define GET_PROPERTY(atom, len) xcb_get_property_unchecked(conn, false, window, atom, XCB_GET_PROPERTY_TYPE_ANY, 0, len)
+#define GET_PROPERTY(atom, len) xcb_get_property(conn, false, window, atom, XCB_GET_PROPERTY_TYPE_ANY, 0, len)
 
     wm_type_cookie = GET_PROPERTY(A__NET_WM_WINDOW_TYPE, UINT32_MAX);
     strut_cookie = GET_PROPERTY(A__NET_WM_STRUT_PARTIAL, UINT32_MAX);

diff --git a/src/x.c b/src/x.c
index 55f447de3b11bf391cc81a1c58a3cd7f9287ed76..5136bf445d6a156628e068861a38c3fec62ec0d7 100644 (file)
--- a/src/x.c
+++ b/src/x.c
@@ -184,7 +184,7 @@ bool window_supports_protocol(xcb_window_t window, xcb_atom_t atom) {
     xcb_icccm_get_wm_protocols_reply_t protocols;
     bool result = false;
 
-    cookie = xcb_icccm_get_wm_protocols_unchecked(conn, window, A_WM_PROTOCOLS);
+    cookie = xcb_icccm_get_wm_protocols(conn, window, A_WM_PROTOCOLS);
     if (xcb_icccm_get_wm_protocols_reply(conn, cookie, &protocols, NULL) != 1)
         return false;
 

diff --git a/src/xcb.c b/src/xcb.c
index 298abd9c855cfd5a6da7d6a02535c609c760e94c..545e2c60494b897d7b222e4efbf2769b8e4f7c7c 100644 (file)
--- a/src/xcb.c
+++ b/src/xcb.c
@@ -254,7 +254,7 @@ void xcb_get_numlock_mask(xcb_connection_t *conn) {
                            XCB_MOD_MASK_5 };
 
     /* Request the modifier map */
-    cookie = xcb_get_modifier_mapping_unchecked(conn);
+    cookie = xcb_get_modifier_mapping(conn);
 
     /* Get the keysymbols */
     keysyms = xcb_key_symbols_alloc(conn);